Re: [PATCH 2/2] gdi32: Added GetGlyphOutlineW tests on glyph that contains empty contour (try3)

[Dmitry Timoshkov]

Piotr Caban <piotr(a)codeweavers.com> wrote:
> create mode 100644 dlls/gdi32/tests/test_empty_contour.sfd
>   create mode 100644 dlls/gdi32/tests/test_empty_contour.ttf

Is it possible to add the desired glyph to an already existing test font?

-- 
Dmitry.

[PATCH 2/2] gdi32: Added GetGlyphOutlineW tests on glyph that contains empty contour (try3)

[Piotr Caban]

On 11/13/12 13:10, Dmitry Timoshkov wrote:
> Piotr Caban <piotr(a)codeweavers.com> wrote:
> > create mode 100644 dlls/gdi32/tests/test_empty_contour.sfd
> >    create mode 100644 dlls/gdi32/tests/test_empty_contour.ttf
>
> Is it possible to add the desired glyph to an already existing test font?

Yes, I was adding it first to wine_test.ttf. In this case 
GetGlyphOutline tests needs to be added in the middle of 
CreateScalableFontResource tests (at least as long as RemoveFontResource 
is not implemented).

I can change it to work this way if it's preferred.

[PATCH 2/2] gdi32: Added GetGlyphOutlineW tests on glyph that contains empty contour (try3)

[Alexandre Julliard]

Piotr Caban <piotr(a)codeweavers.com> writes:
> On 11/13/12 13:10, Dmitry Timoshkov wrote:
> > Piotr Caban <piotr(a)codeweavers.com> wrote:
> > > create mode 100644 dlls/gdi32/tests/test_empty_contour.sfd
> > >    create mode 100644 dlls/gdi32/tests/test_empty_contour.ttf
> >
> > Is it possible to add the desired glyph to an already existing test font?
>
> Yes, I was adding it first to wine_test.ttf. In this case
> GetGlyphOutline tests needs to be added in the middle of
> CreateScalableFontResource tests (at least as long as
> RemoveFontResource is not implemented).
>
> I can change it to work this way if it's preferred.

If it's possible, that's certainly better.

-- 
Alexandre Julliard
julliard(a)winehq.org