Re: [PATCH 1/7] gdiplus: Don't create new instance of IWICBitmapDecoder to encode every frame of animation (try2)

[Vincent Povirk]

@@ -4288,7 +4341,8 @@ static const struct image_codec codecs[NUM_CODECS] = {
             /* SigMask */            emf_sig_mask,
         },
         NULL,
-        decode_image_olepicture_metafile
+        decode_image_olepicture_metafile,
+        select_frame_wic
     },
     {
         { /* WMF */
@@ -4307,7 +4361,8 @@ static const struct image_codec codecs[NUM_CODECS] = {
             /* SigMask */            wmf_sig_mask,
         },
         NULL,
-        decode_image_olepicture_metafile
+        decode_image_olepicture_metafile,
+        select_frame_wic
     },

You have a later patch changing these to NULL, I'm guessing you meant
to do that in this one?

[PATCH 1/7] gdiplus: Don't create new instance of IWICBitmapDecoder to encode every frame of animation (try2)

[Piotr Caban]

On 03/12/15 02:52, Vincent Povirk wrote:
> @@ -4288,7 +4341,8 @@ static const struct image_codec codecs[NUM_CODECS] = {
>               /* SigMask */            emf_sig_mask,
>           },
>           NULL,
> -
>    decode_image_olepicture_metafile
> -
>    decode_image_olepicture_metafile,
> -
>    select_frame_wic
> },
> {
>     { /* WMF */
>
> @@ -4307,7 +4361,8 @@ static const struct image_codec codecs[NUM_CODECS] = {
>               /* SigMask */            wmf_sig_mask,
>           },
>           NULL,
> -
>    decode_image_olepicture_metafile
> -
>    decode_image_olepicture_metafile,
> -
>    select_frame_wic
> },
>
> You have a later patch changing these to NULL, I'm guessing you meant
> to do that in this one?

It doesn't really change anything. I'll change it to NULL here so I 
don't change it twice.

Thanks,
Piotr