winex11: remove unused field

Vincent Povirk madewokherd at gmail.com
Sun Nov 28 20:36:51 CST 2010


I don't know much about this area of Wine, but I think the change is incorrect.

The DC's transform is applied to the font when it is rendered. To do
this correctly, the glyphs must be transformed when they are
rasterized and uploaded.

Note that we only appear to use the first four fields of XFORM (they
are copied to a shorter FMAT2 struct) to rasterize the glyphs. The
last two fields of XFORM only specify an offset, so I think they can
be safely ignored.

2010/11/28 Ričardas Barkauskas <miegalius at gmail.com>:
> Makes Steam store page scrolling smoother.
> Didn't find where this field would be used or what relevant difference
> between fonts it would indicate for this caching.
> Without it lots of fonts differing only by fields in this structure are
> created in such a way that they get marked as unused almost immediately and
> thus cache entries get reused fast.
> Hacking to allow for much more generous cache growth resulted ~880 entries
> (in short run) in font cache and it still managed not to find font in cache
> quite often.
> With this I didn't notice any such growth, no visual changes and no tests
> complained (that don't complain usually).
> Don't know what if any tests I could write for this (or more likely I simply
> wouldn't be able :P).
>
> Rčardas Barkauskas
> REalm
>
>
>
>



More information about the wine-devel mailing list