[PATCH] gdi32/tests: Fix some test failures on newer Win10 releases.
Huw Davies
huw at codeweavers.com
Tue Nov 13 02:40:28 CST 2018
On Mon, Nov 12, 2018 at 08:58:14AM +0300, Nikolay Sivov wrote:
> Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
> ---
> dlls/gdi32/tests/font.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c
> index 0d0770905d..0e895fef11 100644
> --- a/dlls/gdi32/tests/font.c
> +++ b/dlls/gdi32/tests/font.c
> @@ -1003,10 +1003,11 @@ static void test_bitmap_font_metrics(void)
> memset(&gm, 0, sizeof(gm));
> SetLastError(0xdeadbeef);
> ret = GetGlyphOutlineA(hdc, 'A', GGO_METRICS, &gm, 0, NULL, &mat);
> - todo_wine {
> + todo_wine
> ok(ret == GDI_ERROR, "GetGlyphOutline should fail for a bitmap font\n");
> - ok(GetLastError() == ERROR_CAN_NOT_COMPLETE, "expected ERROR_CAN_NOT_COMPLETE, got %u\n", GetLastError());
> - }
> + ret = GetLastError();
> + ok(ret == 0xdeadbeef || broken(ret == ERROR_CAN_NOT_COMPLETE) /* before Win10 */,
> + "Unexpected error %d.\n", ret);
>
I'm not sure I'd classify pre-Win10 behaviour as broken, so just
accepting both without the broken() would be better.
Huw.
More information about the wine-devel
mailing list