gdi32: Add more font substitution tests, make them pass under Wine

Dmitry Timoshkov dmitry at codeweavers.com
Thu Oct 9 00:31:51 CDT 2008


"Jeremy White" <jwhite at winehq.org> wrote:

> This patch has triggered a bug in make test for me; I only notice it
> when I put a Windows flavor of arial.ttf into my windows/fonts directory.
> 
> The specific failure is in get_glyph_indices when we're passing in a
> symbol charset (i.e. the 3rd loop).  I've tracked it to line 3461 of
> gdi32/freetype.c.  Specifically, the lfWeight is set to FW_DONTCARE but
> the charset is Symbol.  Git-blame pointed at this patch.
> 
> I find that if I modify tests/font.c to set a lfWeight of normal in
> get_glyph_indices, then the tests no longer fail.
> 
> However, the test passes on Windows XP without needing that change, so
> I'm afraid that this code must be wrong.
> 
> Here are the full commit details:
> 
> commit a5d288f08c08dc19d217093fdf8622605c92a4e0
> Author: Dmitry Timoshkov <dmitry at codeweavers.com>
> Date:   Tue May 13 22:10:05 2008 +0900
> 
>    gdi32: Add more font substitution tests, make them pass under Wine.
> 
> I was hoping that a resolution to this would be obvious to you, as you
> clearly understood this issue in depth when you wrote the patch.

Thanks for spotting this, I sent a patch which reverts the above change.

-- 
Dmitry.



More information about the wine-devel mailing list