]> git.eshelyaron.com Git - emacs.git/commitdiff
Make ImageMagick multi-image size calculations work for .ico files
authorLars Ingebrigtsen <larsi@gnus.org>
Tue, 24 Sep 2019 17:47:20 +0000 (19:47 +0200)
committerLars Ingebrigtsen <larsi@gnus.org>
Tue, 24 Sep 2019 17:47:20 +0000 (19:47 +0200)
* src/image.c (imagemagick_load_image): Compute image sizes
correctly in multi-image .ico files and the like (bug#18333).

src/image.c

index 06a8154842c7820f8093bc9b8b158c69deae77ee..7cf1bc4c362378e46be502e0f30fa1f00854f862 100644 (file)
@@ -8896,12 +8896,28 @@ imagemagick_load_image (struct frame *f, struct image *img,
      "super-wand". */
   if (MagickGetNumberImages (image_wand) > 1)
     {
-      MagickWand *super_wand = image_wand;
-      image_wand = imagemagick_compute_animated_image (super_wand, ino);
-      if (! image_wand)
-       image_wand = super_wand;
+      /* This is an animated image (it has a delay), so compute the
+        composite image etc. */
+      if (MagickGetImageDelay (image_wand) > 0)
+       {
+         MagickWand *super_wand = image_wand;
+         image_wand = imagemagick_compute_animated_image (super_wand, ino);
+         if (! image_wand)
+           image_wand = super_wand;
+         else
+           DestroyMagickWand (super_wand);
+       }
       else
-       DestroyMagickWand (super_wand);
+       /* This is not an animated image: It's just a multi-image file
+          (like an .ico file).  Just return the correct
+          sub-image.  */
+       {
+         MagickWand *super_wand = image_wand;
+
+         MagickSetIteratorIndex (super_wand, ino);
+         image_wand = MagickGetImage (super_wand);
+         DestroyMagickWand (super_wand);
+       }
     }
 
   /* Retrieve the frame's background color, for use later.  */