Re: [PATCH] gdiplus: Handle some degenerate cases with GdipCreatePath2.

[Nikolay Sivov]

On 4/9/20 10:52 PM, Jeff Smith wrote:
> - if(count <= 0) {
> -
>    *path = NULL;
> -
>    return OutOfMemory;
> - }

Could this be a side effect of treating count as unsigned? By the way, 
actual allocations of point or types arrays are not checked.
> #define expect(expected, got) ok(got == expected, "Expected %.8x, got %.8x\n", expected, got)
>   #define expectf(expected, got) ok(fabs(expected - got) < 2.0, "Expected %.2f, got %.2f\n", expected, got)
> +#define expectp(expected, got) ok(got == expected, "Expected %p, got %p\n", expected, got)
>   #define POINT_TYPE_MAX_LEN (75)

In my opinion we should get rid of such macros, not add more. Especially 
for floating point tests.