Re: gdi32: Add support for loading scalable font resources.

2012-03-30 Thread Dmitry Timoshkov
Huw Davies wrote: > > > +if (size.u.LowPart < sizeof( *dos )) goto fail; > > > > Testing LowPart only could not work for large sizes. > > Of course, but we don't care if very large files fail, they won't be > genuine resource files anyway. Then returning 64-bit sizes doesn't make sence, an

Re: gdi32: Add support for loading scalable font resources.

2012-03-30 Thread Huw Davies
On Fri, Mar 30, 2012 at 07:31:54PM +0900, Dmitry Timoshkov wrote: > Huw Davies wrote: > > > +if (size.u.LowPart < sizeof( *dos )) goto fail; > > Testing LowPart only could not work for large sizes. Of course, but we don't care if very large files fail, they won't be genuine resource files a

Re: gdi32: Add support for loading scalable font resources.

2012-03-30 Thread Dmitry Timoshkov
Huw Davies wrote: > +if (size.u.LowPart < sizeof( *dos )) goto fail; Testing LowPart only could not work for large sizes. -- Dmitry.

Re: [PATCH 3/3] (try2) gdi32: Add support for loading scalable font resources.

2012-03-29 Thread Alexandre Julliard
Huw Davies writes: > --- > dlls/gdi32/font.c | 75 > + > 1 files changed, 75 insertions(+), 0 deletions(-) This one doesn't build: gcc -m32 -c -I. -I. -I../../include -I../../include -I/usr/include/freetype2 -D__WINESRC__ -D_GDI32_ -D_RE