[PATCH] rtworkq/tests: Document the CoGetApartmentType() situation.

Francois Gouget fgouget at codeweavers.com
Fri May 28 11:17:16 CDT 2021


It is reported as missing on Vista but is available on Windows 7 while
rtworkq itself is only available on Windows 8+. So there is no need to
load CoGetApartmentType() dynamically.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
Given that loading CoGetApartmentType() dynamically seems undesirable 
I propose to just document the situation instead.

See:
https://test.winehq.org/data/tests/rtworkq:rtworkq.html
https://www.winehq.org/pipermail/wine-devel/2021-April/185461.html
---
 dlls/rtworkq/tests/rtworkq.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/dlls/rtworkq/tests/rtworkq.c b/dlls/rtworkq/tests/rtworkq.c
index 627e024d761..f6c2fc594e8 100644
--- a/dlls/rtworkq/tests/rtworkq.c
+++ b/dlls/rtworkq/tests/rtworkq.c
@@ -31,7 +31,11 @@ static void test_platform_init(void)
     APTTYPE apttype;
     HRESULT hr;
 
-    /* Startup initializes MTA. */
+    /* Startup initializes MTA.
+     * Note that CoGetApartmentType() is available on Windows 7+ while
+     * rtworkq is only available on Windows 8+. So it is not necessary to
+     * load CoGetApartmentType() dynamically.
+     */
     hr = CoGetApartmentType(&apttype, &qualifier);
     ok(hr == CO_E_NOTINITIALIZED, "Unexpected hr %#x.\n", hr);
 
-- 
2.20.1




More information about the wine-devel mailing list