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:
>     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?