Re: [PATCH v5 0/6] MR4140: uxtheme/tests: Add some tests for OpenThemeData() with various app/class names.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c:
>      "Expected 0xdeadbeef, got 0x%08lx\n",
>      GetLastError());
>
> - SetLastError(0xdeadbeef);
> - bTPDefined = IsThemePartDefined(hTheme, 0 , 0);
> - todo_wine
> - ok( bTPDefined == FALSE, "Expected FALSE\n" );
> - ok( GetLastError() == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got 0x%08lx\n", GetLastError() );
> -

I am not sure of the exact intent of the original commit d9c5cef339c7e169e6b9dbd96a3158f91fa9056c. But it seems to me here IsThemePartDefined() is deliberately tested after the theme handle is closed. Similar to the GetWindowTheme(). So I would prefer not to move this.