Re: [PATCH v3 0/6] MR3302: Draft: gdiplus: Fix GdipGetPropertySize and GdipGetAllPropertyItems return values when no properties exist.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/tests/image.c:
> status = GdipGetPropertySize(image, &prop_size, &prop_count);
>          expect(expected, status);
>
> +        prop_item = HeapAlloc(GetProcessHeap(), 0, prop_size);
> +        status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL);
> +        expect(InvalidParameter, status);
> +        status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL);
> +        expect(InvalidParameter, status);
> +        status = GdipGetAllPropertyItems(image, 0, 0, NULL);
> +        expect(InvalidParameter, status);
> +        expected = (image_type == ImageTypeMetafile) ? NotImplemented : InvalidParameter;
> +        broken_ok = (td[i].prop_count == 1 || td[i].prop_count2 == 1);
> +        status = GdipGetAllPropertyItems(image, prop_size, 1, prop_item);
> +        ok(status == expected || broken(status == Ok && broken_ok), /* XP */
> +                "Expected %d, got %d\n", expected, status);

I don't think this is really broken if `prop_count` happens to be 1?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3302#note_39086