Re: [PATCH 0/1] MR4531: uxtheme/tests: Add some tests for OpenThemeFile.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c:
> -
>    return;
> - }
> -
> - ls = RegGetValueW(HKEY_CURRENT_USER,
> -
>                      L"Software\\Microsoft\\Windows\\CurrentVersion\\ThemeManager", L"DllName",
> -
>                      RRF_RT_REG_SZ, NULL, currentThemePath, &pathSize);
> - if (ls == ERROR_FILE_NOT_FOUND)
> - {
> -
>    win_skip("DllName registry value not found.\n");
> -
>    return;
> - }
> - ok(ls == ERROR_SUCCESS, "RegGetValueW failed: %ld\n", ls);
> -
> - htf = (void *)0xdeadbeef;
> - hr = pOpenThemeFile(NULL, NULL, NULL, &htf, 0);
> - todo_wine ok(hr == E_POINTER, "Expected E_POINTER, got 0x%08lx\n", hr);

Are you going to send a fix or it's just the test? Because if you have a fix then it's better to send it together.