[PATCH 2/3] gdiplus: Convert gif frame to DIB on load

Piotr Caban piotr.caban at gmail.com
Tue Mar 10 06:53:16 CDT 2015


On 03/10/15 02:18, Vincent Povirk wrote:
> I don't like that we essentially have another instance here of the
> bitmap create/destroy logic. I think I'd rather we create another
> bitmap than use the existing one with new image data.
>
> It's not clear to me that decode_image_wic is even worth calling. Most
> of the logic in there is for reading image bits, and couldn't we in
> theory use blit_gif_frame or select_image_gif from the next patch for
> that?
>
> Or, if we wanted to add another hook to it, we could ask
> decode_image_wic to not attempt to find a matching format, and instead
> fall back on the conversion to 32bppARGB.
I'll add additional parameter to decode_image_wic so it can be reused here.



More information about the wine-devel mailing list