diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2020-05-29 09:01:07 +0200 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2020-05-29 10:41:24 +0200 |
| commit | c1aab4ca12f8e3b68240d064d82cf72ccb527648 (patch) | |
| tree | a80f04af627171fcff3ed45512452f39a22bf855 /dtrans | |
| parent | 31b0be0f21479323408e128f2e8a1a795e037e74 (diff) | |
Prefer CoInitializeEx instead of CoInitialize (2)
As https://docs.microsoft.com/fr-fr/windows/win32/api/objbase/nf-objbase-coinitialize?redirectedfrom=MSDN
advised.
Moreover, it'll make concurrency model explicit
Change-Id: I8eaa7c0ba4e0c5131b178596cf5310df92ff41ce
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95093
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Diffstat (limited to 'dtrans')
| -rw-r--r-- | dtrans/source/win32/clipb/MtaOleClipb.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/dtrans/source/win32/clipb/MtaOleClipb.cxx b/dtrans/source/win32/clipb/MtaOleClipb.cxx index 70fce3bc64ad..0e57b5ab1442 100644 --- a/dtrans/source/win32/clipb/MtaOleClipb.cxx +++ b/dtrans/source/win32/clipb/MtaOleClipb.cxx @@ -200,10 +200,10 @@ public: { /* we only call CoUninitialize when - CoInitialize returned S_FALSE, what + CoInitializeEx returned S_FALSE, what means that com was already initialize for that thread so we keep the balance - if CoInitialize returned S_OK what means + if CoInitializeEx returned S_OK what means com was not yet initialized we better let com initialized or we may run into the realm of undefined behaviour @@ -678,7 +678,7 @@ unsigned int WINAPI CMtaOleClipboard::clipboardChangedNotifierThreadProc( LPVOID CMtaOleClipboard* pInst = static_cast< CMtaOleClipboard* >( pParam ); OSL_ASSERT( nullptr != pInst ); - CoInitialize( nullptr ); + CoInitializeEx( nullptr, COINIT_APARTMENTTHREADED ); // assuming we don't need a lock for // a boolean variable like m_bRun... |