Subject: Re: bug#66221: 29.1; Animated webp image leaks memory

From: Stefan Kangas <[email-masked]>
Date: Fri, 29 Sep 2023 08:42:34 -0700

Eli Zaretskii <[email-masked]> writes:

> Feel free to step through clear-image-cache and see what happens
> there.

I noticed that `clear-image-cache' won't clear the rest of the image
cache if we have a non-empty animation_cache.  Is that expected?

IOW, would a change like the following make more sense?

diff --git a/src/image.c b/src/image.c
index 84db9bfb3b8..e98768e70bd 100644
--- a/src/image.c
+++ b/src/image.c
@@ -2345,7 +2345,6 @@ DEFUN ("clear-image-cache", Fclear_image_cache,
Sclear_image_cache,
 #if defined (HAVE_WEBP) || defined (HAVE_GIF)
       anim_prune_animation_cache (XCDR (animation_cache));
 #endif
-      return Qnil;
     }

   if (! (NILP (filter) || FRAMEP (filter)))