]> git.eshelyaron.com Git - emacs.git/commitdiff
Fix previous commit.
authorLars Magne Ingebrigtsen <larsi@gnus.org>
Mon, 19 Aug 2013 15:14:42 +0000 (17:14 +0200)
committerLars Magne Ingebrigtsen <larsi@gnus.org>
Mon, 19 Aug 2013 15:14:42 +0000 (17:14 +0200)
(imagemagick_get_animation_cache): Revert to previous definition
so that it actually works.  But keep the memory leak fix.

src/image.c

index ec89ee076d9cd3d97e48864c4cad1ae3144562fa..00f2a035c443f92913836324d8623e7cc7eb7162 100644 (file)
@@ -7895,6 +7895,7 @@ imagemagick_create_cache (char *signature)
   cache->wand = 0;
   cache->index = 0;
   cache->next = 0;
+  cache->update_time = current_emacs_time ();
   return cache;
 }
 
@@ -7927,28 +7928,25 @@ imagemagick_get_animation_cache (MagickWand *wand)
 {
   char *signature = MagickGetImageSignature (wand);
   struct animation_cache *cache;
-  struct animation_cache **pcache;
 
   imagemagick_prune_animation_cache ();
+  cache = animation_cache;
 
-  if (! animation_cache)
-    animation_cache = cache = imagemagick_create_cache (signature);
-  else
+  if (! cache)
     {
-      for (pcache = &animation_cache; *pcache; pcache = &cache->next)
-       {
-         cache = *pcache;
-         if (! cache)
-           {
-             animation_cache = cache = imagemagick_create_cache (signature);
-             break;
-           }
-         if (strcmp (signature, cache->signature) == 0)
-           {
-             DestroyString (signature);
-             break;
-           }
-       }
+      animation_cache = imagemagick_create_cache (signature);
+      return animation_cache;
+    }
+
+  while (strcmp(signature, cache->signature) &&
+        cache->next)
+    cache = cache->next;
+
+  if (strcmp(signature, cache->signature))
+    {
+      cache->next = imagemagick_create_cache (signature);
+      DestroyString (signature);
+      return cache->next;
     }
 
   cache->update_time = current_emacs_time ();