flint++ static analyzer detected items (not all that were likely legit positives, but a start) as well as some clang detected unused vars, cleanup_

external/3rd/library/soePlatform/ChatAPI/utils/Base/ScopeLock.cpp

@@ -16,7 +16,7 @@ namespace Base
         mMutex->Lock();
     }
 
-    CScopeLock::CScopeLock(CScopeLock& lock) :
+    CScopeLock::CScopeLock(const CScopeLock& lock) :
         mMutex(lock.mMutex)
     {
         mMutex->Lock();