bearermanagement/mpm/src/mpmserver.cpp
branchRCL_3
changeset 45 4c83dcfb6f1a
parent 36 04408506c6e0
child 55 fc7b30ed2058
--- a/bearermanagement/mpm/src/mpmserver.cpp	Wed Jun 09 10:07:12 2010 +0300
+++ b/bearermanagement/mpm/src/mpmserver.cpp	Mon Jun 21 16:06:29 2010 +0300
@@ -701,26 +701,25 @@
     }
 
 // -----------------------------------------------------------------------------
-// CMPMServer::CheckIfStarted
+// CMPMServer::CheckUsageOfIap
 // -----------------------------------------------------------------------------
 //
-TBool CMPMServer::CheckIfStarted( const TUint32 aIapId , 
-                                  const TConnectionId aConnId )
+TConnectionState CMPMServer::CheckUsageOfIap( const TUint32 aIapId , 
+                                  const TConnectionId aOwnConnId )
     {
-    MPMLOGSTRING3( "CMPMServer::CheckIfStarted - aIapId = %i, aConnId = 0x%x", 
-                  aIapId, aConnId )
+    MPMLOGSTRING3( "CMPMServer::CheckUsageOfIap - aIapId = %i, aOwnConnId = 0x%x", 
+                  aIapId, aOwnConnId )
 
-    TConnectionState state( EIdle );
-    TBool stopLoop( EFalse );
+    TConnectionState state = EIdle;
 
     // Loop all connections until EStarted is found or no more connections
     // 
-    for ( TInt i = 0; ( ( i < iActiveBMConns.Count() ) && !stopLoop ); i++ )
+    for ( TInt i = 0; i < iActiveBMConns.Count(); i++ )
         {
         // Check if IAP Id matches; exclude matching with own connection
         // 
         if ( iActiveBMConns[i].iConnInfo.iIapId == aIapId && 
-             iActiveBMConns[i].iConnInfo.iConnId != aConnId)
+             iActiveBMConns[i].iConnInfo.iConnId != aOwnConnId)
             {
             state = iActiveBMConns[i].iConnInfo.iState;  
 
@@ -728,7 +727,7 @@
             // 
             if ( state == EStarted )
                 {
-                stopLoop = ETrue;
+                break; // Breaks the for loop
                 }
             }
         }
@@ -738,44 +737,33 @@
         {
         case EStarting:
             {
-            MPMLOGSTRING( "CMPMServer::CheckIfStarted - Starting" )
+            MPMLOGSTRING( "CMPMServer::CheckUsageOfIap - Starting" )
             break;
             }
         case EStarted:
             {
-            MPMLOGSTRING( "CMPMServer::CheckIfStarted - Started" )
+            MPMLOGSTRING( "CMPMServer::CheckUsageOfIap - Started" )
             break;
             }
         case EIdle:
             {
-            MPMLOGSTRING( "CMPMServer::CheckIfStarted - Idle" )
+            MPMLOGSTRING( "CMPMServer::CheckUsageOfIap - Idle" )
             break;
             }
         case ERoaming:
             {
-            MPMLOGSTRING( "CMPMServer::CheckIfStarted - Roaming" )
+            MPMLOGSTRING( "CMPMServer::CheckUsageOfIap - Roaming" )
             break;
             }
         default:
             {
-            MPMLOGSTRING( "CMPMServer::CheckIfStarted - Unknown" )
+            MPMLOGSTRING( "CMPMServer::CheckUsageOfIap - Unknown" )
             break;
             }
         }
 #endif // _DEBUG
         
-    //Return true incase the matching connection is in EStarting state also because
-    //sometimes when connections are started simultaneously (for same iapID) 
-    //the first connection may still be in EStarting state. 
-    //
-    if ( state == EStarted || state == EStarting )
-        {
-        return ETrue;
-        }
-    else
-        {
-        return EFalse;
-        }
+    return state;
     }
 
 // -----------------------------------------------------------------------------