From: Richard M. Stallman <rms@gnu.org>
Date: Wed, 19 Jul 1995 15:47:28 +0000 (+0000)
Subject: (unload_color): Don't free colors if immutable colormap.
X-Git-Tag: emacs-19.34~3251
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=828e66d1a5d6e6a50d6fa207e29b2a6374bb81c8;p=emacs.git

(unload_color): Don't free colors if immutable colormap.
---

diff --git a/src/xfaces.c b/src/xfaces.c
index 8872b724810..59e09d95077 100644
--- a/src/xfaces.c
+++ b/src/xfaces.c
@@ -363,14 +363,22 @@ unload_color (f, pixel)
 {
   Colormap cmap;
   Display *dpy = FRAME_X_DISPLAY (f);
+  int class = FRAME_X_DISPLAY_INFO (f)->visual->class;
+
   if (pixel == FACE_DEFAULT
       || pixel == BLACK_PIX_DEFAULT (f)
       || pixel == WHITE_PIX_DEFAULT (f))
     return;
   cmap = DefaultColormapOfScreen (DefaultScreenOfDisplay (dpy));
-  BLOCK_INPUT;
-  XFreeColors (dpy, cmap, &pixel, 1, (unsigned long)0);
-  UNBLOCK_INPUT;
+  
+  /* If display has an immutable color map, freeing colors is not
+     necessary and some servers don't allow it.  So don't do it.  */
+  if (! (class == StaticColor || class == StaticGray || class == TrueColor))
+    {
+      BLOCK_INPUT;
+      XFreeColors (dpy, cmap, &pixel, 1, (unsigned long)0);
+      UNBLOCK_INPUT;
+    }
 }
 
 DEFUN ("pixmap-spec-p", Fpixmap_spec_p, Spixmap_spec_p, 1, 1, 0,