diff -r 822a45792fdd -r e1d68407ed06 wlanutilities/wpswizard/src/cwpsactiverunner.cpp --- a/wlanutilities/wpswizard/src/cwpsactiverunner.cpp Fri Sep 03 09:37:37 2010 +0300 +++ b/wlanutilities/wpswizard/src/cwpsactiverunner.cpp Thu Sep 16 12:52:33 2010 +0300 @@ -91,23 +91,13 @@ OstTraceFunctionEntry1( CWPSACTIVERUNNER_CONSTRUCTL_ENTRY, this ); CActiveScheduler::Add(this); - InitializeL(); - OstTraceFunctionExit1( CWPSACTIVERUNNER_CONSTRUCTL_EXIT, this ); -} + + iWLANMgmtClient = CWlanMgmtClient::NewL(); -/*! - * Initializes the member variables for making the middleware calls - * - */ - -void CWpsActiveRunner::InitializeL() -{ - OstTraceFunctionEntry1( CWPSACTIVERUNNER_INITIALIZEL_ENTRY, this ); - iWLANMgmtClient = CWlanMgmtClient::NewL(); iIapParametersArray = new (ELeave) CArrayFixSeg< TWlanProtectedSetupCredentialAttribute> (KArrayGranularity); - OstTraceFunctionExit1( CWPSACTIVERUNNER_INITIALIZEL_EXIT, this ); + OstTraceFunctionExit1( CWPSACTIVERUNNER_CONSTRUCTL_EXIT, this ); } /*! @@ -139,10 +129,10 @@ ssid.Copy(aSsid); TBuf8 pinCode; + // When Pin is zero it means that push-button mode is used. if (aPin == 0) { pinCode.AppendFill('0', 8); - } - else { + } else { pinCode.AppendNum(aPin); } @@ -165,22 +155,19 @@ TInt completionCode = iStatus.Int(); QList credentials; - if(!isCancelTriggered) { + if (!isCancelTriggered) { if (completionCode < KErrNone) { //Raise Error - QT_TRYCATCH_LEAVING(iObserver.WpsActiveRunnerStopped(credentials, completionCode)); - } - else { + QT_TRYCATCH_LEAVING(iObserver.WpsActiveRunnerStopped(credentials, completionCode)); + } else { TInt len = iIapParametersArray->Length(); TInt count; - for(count=0;countCount();count++) - { + for (count = 0 ; count < iIapParametersArray->Count() ; count++) { TWlanProtectedSetupCredentialAttribute attr =(*iIapParametersArray)[count]; credentials.append(attr); - } + } QT_TRYCATCH_LEAVING(iObserver.WpsActiveRunnerStopped(credentials,completionCode)); - } } @@ -194,10 +181,9 @@ { OstTraceFunctionEntry1( CWPSACTIVERUNNER_DOCANCEL_ENTRY, this ); isCancelTriggered = true; - if(iWLANMgmtClient) - { - iWLANMgmtClient->CancelProtectedSetup(); - } + + iWLANMgmtClient->CancelProtectedSetup(); + OstTraceFunctionExit1( CWPSACTIVERUNNER_DOCANCEL_EXIT, this ); } @@ -211,13 +197,10 @@ OstTrace1( TRACE_ERROR, CWPSACTIVERUNNER_RUNERROR, "CWpsActiveRunner::RunError;aError=%d", aError ); QList credentials; - if(iWLANMgmtClient) - { - iWLANMgmtClient->CancelProtectedSetup(); - } - + iWLANMgmtClient->CancelProtectedSetup(); + QT_TRYCATCH_LEAVING(iObserver.WpsActiveRunnerStopped(credentials, aError)); - return 0; + return KErrNone; }