March 7, 2026
4:11 a.m.
On Wed Mar 4 14:00:39 2026 +0000, Nikolay Sivov wrote:
This does not look good in my opinion. To me it looks like checking for some non obvious side effect. If you need to check current-or-mta case, you can use InternalIsProcessInitialized() from combase. We already use it in ole32. The RegisterDragDrop function requires OleInitialize to be called beforehand, and this check ensures that the resources initialized by OleInitialize are still allocated and have not been released.
In the v2 patch, I added an InternalIsProcessInitialized call to explicitly verify that COM have been initialized by OleInitialize or CoInitializeEx. Thanks for reviewing. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10245#note_131534