Re: [PATCH v2 0/1] MR3581: gdiplus: add support for GdipPathIterNextPathType

[Jeffrey Smith (＠whydoubt)]

Jeffrey Smith (@whydoubt) commented about dlls/gdiplus/tests/pathiterator.c:
> GdipCreatePath(FillModeAlternate, &path);
>      GdipCreatePathIter(&iter, path);
>
> -    result = -2;
> +    result = start = end = (INT)0xdeadbeef;
>      closed = TRUE;
>      stat = GdipPathIterNextSubpath(iter, &result, &start, &end, &closed);
>      expect(Ok, stat);
>      expect(0, result);
> +    expect((INT)0xdeadbeef, start);
> +    expect((INT)0xdeadbeef, end);
> +    expect(TRUE, closed); /* Not changed */

FWIW, to _really_ test that it wasn't changed, it seems better to set it to a more-unlikely value.  Since BOOL is actually an int, you could set it to and check for -1 or 0xdeadbeef.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3581#note_42509