diff options
author | Chong Yidong <cyd@stupidchicken.com> | 2007-08-07 16:25:26 +0000 |
---|---|---|
committer | Chong Yidong <cyd@stupidchicken.com> | 2007-08-07 16:25:26 +0000 |
commit | 8198352bca8b704e80bead200a790e0bc616723b (patch) | |
tree | 446d0fb9214033ce233b5af4db7a621f8198cc8c | |
parent | 8a7e2b234935c9c7eabab2538e34ac1f56d15766 (diff) | |
download | emacs-8198352bca8b704e80bead200a790e0bc616723b.tar.gz |
Minor cleanup.
-rw-r--r-- | src/image.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/image.c b/src/image.c index fb0420045e2..a3cd3195217 100644 --- a/src/image.c +++ b/src/image.c @@ -1644,6 +1644,8 @@ search_image_cache (f, spec, hash) struct image_cache *c = FRAME_X_IMAGE_CACHE (f); int i = hash % IMAGE_CACHE_BUCKETS_SIZE; + if (!c) return NULL; + /* If the image spec does not specify a background color, the cached image must have the same background color as the current frame. The foreground color must also match, for the sake of monochrome @@ -1655,13 +1657,10 @@ search_image_cache (f, spec, hash) for formats that don't use transparency (such as jpeg), or if the image spec specifies :background. However, the extra memory usage is probably negligible in practice, so we don't bother. */ - if (!c) return NULL; for (img = c->buckets[i]; img; img = img->next) if (img->hash == hash && !NILP (Fequal (img->spec, spec)) - /* If the image spec specifies a background, it doesn't matter - what the frame background is. */ && img->frame_foreground == FRAME_FOREGROUND_PIXEL (f) && img->frame_background == FRAME_BACKGROUND_PIXEL (f)) break; @@ -6360,9 +6359,8 @@ png_load (f, img) } } /* The commented-out code checked if the png specifies a default - background color, and uses that. Since we rely on the - current frame background, it is actually OK for us to ignore - this part. + background color, and uses that. Since we use the current + frame background, it is OK for us to ignore this. else if (fn_png_get_bKGD (png_ptr, info_ptr, &image_bg)) fn_png_set_background (png_ptr, image_bg, |