Re: [PATCH v3 0/2] MR9287: uxtheme: Handle NULL options in DrawThemeTextEx().

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c:
> +    hdc = GetDC(hwnd);
> +    SetRect(&rect, 0, 0, 1, 1);
> +
> +    hr = DrawThemeTextEx(NULL, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, NULL);
> +    ok(hr == E_HANDLE, "Got unexpected hr %#lx.\n", hr);
> +
> +    hr = DrawThemeTextEx(htheme, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, NULL);
> +    ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);
> +
> +    options.dwSize = sizeof(options);
> +    options.dwFlags = DTT_VALIDBITS;
> +    options.iFontPropId = TMT_BODYFONT;
> +    hr = DrawThemeTextEx(htheme, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, &options);
> +    ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);
> +
> +    options.crText = CLR_INVALID;
> Same here. You should specify DTT_TEXTCOLOR.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9287#note_120702