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

[Bartosz Kosiorek (＠gang65)]

On Wed Aug 16 19:29:46 2023 +0000, Jeffrey Smith wrote:
> - If you swap the if/else blocks, the current if body doesn't need to be
> in a block:
> ```suggestion:-8+0
>     if ((iter->pathdata.Types[i] & PathPointTypePathTypeMask) != PathPointTypeStart)
>         return Ok;
>     *start = i;
>     i++;
>     *end = i;
>     *type = iter->pathdata.Types[i] & PathPointTypePathTypeMask;
>     i++;
> ```
> - Why do you not set *result when you return?
> How it could be done?

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