Huw Davies (@huw) commented about dlls/ole32/tests/marshal.c:
- OXID second_oxid;
- cLocks = 0;
- external_connections = 0;
- CoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
- first_oxid = get_apartment_oxid();
- CoUninitialize();
- CoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
- second_oxid = get_apartment_oxid();
- CoUninitialize();
- ok(first_oxid != second_oxid, "Re-created apartment has old OXID: %s\n", wine_dbgstr_longlong(first_oxid));
+}
I think it would make sense to also test the `COINT_MULTITHREADED` case, perhaps by calling this function twice with a `flags` parameter. It would also make sense to split the tests off to a separate commit, before the implementation change, with appropriate `todo_wine`s which would get removed in the second commit.