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

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/uxtheme/draw.c:
> -
>        FIXME("unsupported flags 0x%08lx\n", options->dwFlags);
> -
>    if (options->dwFlags & DTT_FONTPROP)
> -
>        fontProp = options->iFontPropId;
> -
>    if (options->dwFlags & DTT_TEXTCOLOR)
> -
>        textColor = options->crText;
> - }
> - if (options->dwFlags & DTT_FONTPROP)
> -
>    fontProp = options->iFontPropId;
> - else
> -
>    fontProp = TMT_FONT;
> - if (textColor == CLR_INVALID)
> - {
> -
>    if(FAILED(GetThemeColor(hTheme, iPartId, iStateId, TMT_TEXTCOLOR, &textColor)))
> -
>        textColor = GetTextColor(hdc);
> - }

Existing code seems wrong already. TMT_FONT and TMT_TEXTCOLOR are not properties, but property types. But this change makes it worse with CLR_INVALID.