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

[Jeffrey Smith (＠whydoubt)]

Jeffrey Smith (@whydoubt) commented about dlls/gdiplus/tests/pathiterator.c:
> +    result = start = end = (INT)0xdeadbeef;
> +    closed = TRUE;
> +    stat = GdipPathIterNextSubpath(iter, &result, &start, &end, &closed);
> +    expect(Ok, stat);
> +    expect(13, result);
> +    expect(6, start);
> +    expect(18, end);
> +    expect(TRUE, closed);
> +
> +    /* When subPath is set to Ellipse figure, the type is PathPointTypeBezier */
> +    stat = GdipPathIterNextPathType(iter, &result, &type, &start, &end);
> +    expect(Ok, stat);
> +    expect(6, start);
> +    expect(18, end);
> +    expect(PathPointTypeBezier, type);
> +    expect(13, result);
- Seems more consistent to check these in the order: result, type, start, end.
- You didn't initialize those values before the call.

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