Re: [PATCH v3 0/5] MR3621: GdipCreateBitmapFromICON fixes part 1/2

[Bartosz Kosiorek (＠gang65)]

Bartosz Kosiorek (@gang65) commented about dlls/gdiplus/tests/image.c:
> -       stat = GdipGetImageType((GpImage*)bitmap, &type);
> -       expect(Ok, stat);
> -       expect(ImageTypeBitmap, type);
> -       stat = GdipGetImagePixelFormat((GpImage*)bitmap, &format);
> -       expect(Ok, stat);
> -       expect(PixelFormat32bppARGB, format);
> -       /* raw format */
> +       expect_image_properties((GpImage*)bitmap, 16, 16, __LINE__);
>         expect_rawformat(&ImageFormatMemoryBMP, (GpImage*)bitmap, __LINE__, FALSE);
>         GdipDisposeImage((GpImage*)bitmap);
>      }
>      DestroyIcon(hIcon);
>
> -    /* color icon 8 bpp */
> -    hbmMask = CreateBitmap(16, 16, 1, 8, bmp_bits);
> +    /* monochrome cursor */
> The comment is not valid here. We are testing here what will happen if `info.fIcon = FALSE`

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3621#note_43153