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?