[CICERO][MSCTF] ITfContext: Follow-up of #8224 (#8231)

Follow-up of #8224.
JIRA issue: CORE-19361
- Add cicIsNullPtr function in Cicero <cicbase.h>.
- Reordering of CMakeLists.txt items.
- Minor refactoring.

base/ctf/cicero/cicbase.h

@@ -30,6 +30,11 @@ static inline void cicMemFree(LPVOID ptr)
         LocalFree(ptr);
 }
 
+static inline bool cicIsNullPtr(LPCVOID ptr)
+{
+    return !ptr;
+}
+
 struct CicNoThrow { };
 #define cicNoThrow CicNoThrow{}
 

base/ctf/msctf/CMakeLists.txt

@@ -13,8 +13,8 @@ list(APPEND SOURCE
 
 list(APPEND PCH_SKIP_SOURCE
     categorymgr.cpp
-    context.cpp
     compartmentmgr.cpp
+    context.cpp
     displayattributemgr.cpp
     documentmgr.cpp
     langbarmgr.cpp

base/ctf/msctf/compartmentmgr.cpp

@@ -26,11 +26,6 @@
 #include <wine/debug.h>
 WINE_DEFAULT_DEBUG_CHANNEL(msctf);
 
-static inline bool cicIsNullPtr(LPCVOID ptr)
-{
-    return !ptr;
-}
-
 ////////////////////////////////////////////////////////////////////////////
 
 typedef struct tagCompartmentValue

base/ctf/msctf/context.cpp

@@ -26,18 +26,17 @@
 #include <wine/debug.h>
 WINE_DEFAULT_DEBUG_CHANNEL(msctf);
 
-static inline bool cicIsNullPtr(LPCVOID ptr)
-{
-    return !ptr;
-}
-
 ////////////////////////////////////////////////////////////////////////////
 
 class CContext
     : public ITfContext
     , public ITfSource
+    // , public ITfContextComposition
     , public ITfContextOwnerCompositionServices
+    // , public ITfContextOwnerServices
     , public ITfInsertAtSelection
+    // , public ITfMouseTracker
+    // , public ITfQueryEmbedded
     , public ITfSourceSingle
     , public ITextStoreACPSink
     , public ITextStoreACPServices
@@ -412,16 +411,14 @@ STDMETHODIMP CContext::GetSelection(
         hr = m_pITextStoreACP->GetSelection(ulIndex + i, 1, &acps, &fetched);
         if (hr == TS_E_NOLOCK)
             return TF_E_NOLOCK;
-        else if (SUCCEEDED(hr))
-        {
+        else if (FAILED(hr))
+            break;
+
         pSelection[totalFetched].style.ase = (TfActiveSelEnd)acps.style.ase;
         pSelection[totalFetched].style.fInterimChar = acps.style.fInterimChar;
         Range_Constructor(this, m_pITextStoreACP, cookie->lockType, acps.acpStart, acps.acpEnd, &pSelection[totalFetched].range);
         totalFetched++;
     }
-        else
-            break;
-    }
 
     *pcFetched = totalFetched;
     return hr;