[PATCH] comctl32: Use correct font for themed headers. (try 3)

Nikolay Sivov nsivov at codeweavers.com
Mon Mar 23 14:51:51 CDT 2015


On 03/23/2015 10:44 PM, Mark Harmstone wrote:

Hey, looks much better, thanks.

One minor thing left, and I'm sorry that I missed it last time:

> +	    if (theme) {
> +		DrawThemeText(theme, hdc, HP_HEADERITEM, state, phdi->pszText,
> +			    lstrlenW(phdi->pszText), DT_LEFT|DT_END_ELLIPSIS|DT_VCENTER|DT_SINGLELINE,
> +			    0, &r);
> +	    } else {
> +

Will it work as docs claim if you simply pass -1 as a text length? 
Otherwise looks good to me.



More information about the wine-devel mailing list