Revision: 201033 RCL_3 PDK_3.0.4
authorDremov Kirill (Nokia-D-MSW/Tampere) <kirill.dremov@nokia.com>
Wed, 13 Oct 2010 15:47:46 +0300
branchRCL_3
changeset 42 9db35237e6aa
parent 38 d9641c85af2b
Revision: 201033 Kit: 201041
webservices/wscore/src/senxmldao.cpp
--- a/webservices/wscore/src/senxmldao.cpp	Tue Sep 14 23:25:14 2010 +0300
+++ b/webservices/wscore/src/senxmldao.cpp	Wed Oct 13 15:47:46 2010 +0300
@@ -1712,16 +1712,16 @@
         // when server (main thread) goes down.
         
         pLookupInfo = NULL;
-        if(pHostlet && pHostlet->Threadsafe())
+        if(pHostlet->Threadsafe())
             {
-            TLSLOG_L(KSenCoreServiceManagerLogChannelBase  , KMinLogLevel,"- Adding new threadsafe, unsharable provider into cache.");
+            TLSLOG_L(KSenCoreServiceManagerLogChannelBase  , KMaxLogLevel,"- Adding new threadsafe, unsharable provider into cache.");
             pLookupInfo = CSenHostletLookupInfo::NewLC(aReqThreadId, aReqConsumerId);
             }
         else
             {
             // Any non-threadsafe provider should compare whether consumer ID
             // is equal. The thread ID is irrelevant in the matching.
-            TLSLOG_L(KSenCoreServiceManagerLogChannelBase  , KMinLogLevel,"- Adding new non-threadsafe, unsharable provider into cache.");
+            TLSLOG_L(KSenCoreServiceManagerLogChannelBase  , KMaxLogLevel,"- Adding new non-threadsafe, unsharable provider into cache.");
             pLookupInfo = CSenHostletLookupInfo::NewLC(KNullDesC, aReqConsumerId);
             }
 
@@ -1729,7 +1729,7 @@
         append = iUnsharableProviders.Append(pLookupInfo, pHostlet);
         if(append==KErrNone)
             {
-            TLSLOG_FORMAT((KSenCoreServiceManagerLogChannelBase  , KMinLogLevel, _L8("- Loaded new unsharable hostlet instance: 0x%X, lookup info: 0x%X, lookup count: %d"), pHostlet, pLookupInfo, pLookupInfo->LookupCount()));
+            TLSLOG_FORMAT((KSenCoreServiceManagerLogChannelBase  , KMaxLogLevel, _L8("- Loaded new unsharable hostlet instance: 0x%X, lookup info: 0x%X, lookup count: %d"), pHostlet, pLookupInfo, pLookupInfo->LookupCount()));
             CleanupStack::Pop(2); // pLookupInfo, pHostlet
             }
         else
@@ -1933,26 +1933,12 @@
 
 TPtrC CSenHostletLookupInfo::ThreadId() const
     {
-    if(ipFullThreadName)
-    	{
-    	return *ipFullThreadName;
-    	}
-    else
-    	{
-    	return KNullDesC();
-    	}
+    return *ipFullThreadName;
     }
 
 TPtrC8 CSenHostletLookupInfo::ConsumerId() const
     {
-    if(ipUniqueConsumerId)
-    	{
-    	return *ipUniqueConsumerId;
-    	}
-    else
-    	{
-    	return KNullDesC8();
-    	}	
+    return *ipUniqueConsumerId;
     }
 
 
@@ -1974,15 +1960,9 @@
 
 TBool CSenHostletLookupInfo::operator==(const CSenHostletLookupInfo& aHostletRequestor)
     {
-    if(ipFullThreadName && ipUniqueConsumerId)
-    	{
-    	return((ipFullThreadName->Length()==0 || aHostletRequestor.ThreadId() == *ipFullThreadName)
+    return((ipFullThreadName->Length()==0 || aHostletRequestor.ThreadId() == *ipFullThreadName)
             && aHostletRequestor.ConsumerId() == *ipUniqueConsumerId);
-    	}
-    else
-    	{
-    	return EFalse;
-    	}	
+    
     //    return (aHostletRequestor.ThreadId() == *ipFullThreadName
 //            && aHostletRequestor.ConsumerId() == *ipUniqueConsumerId);
     }