Re: RESEND: Page Fault With Freetype calls fixed

[Alexandre Julliard]

Rein Klazes <wijn(a)wanadoo.nl> writes:
> Not included in the new release, no comments. Is there a problem with
> this one ?

I think it would be cleaner to have WineEngGetFontData do the right
thing directly, instead of calling it twice.

-- 
Alexandre Julliard
julliard(a)winehq.org

RESEND: Page Fault With Freetype calls fixed

[Rein Klazes]

On Tue, 29 Nov 2005 17:37:21 +0100, you wrote:
> Rein Klazes <wijn(a)wanadoo.nl> writes:
> > Not included in the new release, no comments. Is there a problem with
> > this one ?
>
> I think it would be cleaner to have WineEngGetFontData do the right
> thing directly, instead of calling it twice.

My idea was that since it is an internal function, and since this is the
only call (out of 7) that is interested in the returned count, not to
modify the function.

Here is a corrected patch. 

Zach: reversing the old patch and testing this one would be appreciated.

Changelog:
dlls/gdi	: freetype.c
Make WineEngGetFontData always return the used byte count.

Rein.