X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=render%2Fcolor.c;h=c672a6d275a456f2b6fe17b15a62d09384589b47;hb=34446063fa6f954f81200f2778c099865b374d2c;hp=5fdce39148b9f7722cd0c1ba02e192a9863bca6d;hpb=ac9d8c58cb2286a98f699fe1b98b56e11c262868;p=chaz%2Fopenbox

diff --git a/render/color.c b/render/color.c
index 5fdce391..c672a6d2 100644
--- a/render/color.c
+++ b/render/color.c
@@ -1,3 +1,22 @@
+/* -*- indent-tabs-mode: nil; tab-width: 4; c-basic-offset: 4; -*-
+
+   color.c for the Openbox window manager
+   Copyright (c) 2003        Ben Jansens
+   Copyright (c) 2003        Derek Foreman
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   See the COPYING file for a copy of the GNU General Public License.
+*/
+
 #include "render.h"
 #include "color.h"
 #include "instance.h"
@@ -59,18 +78,17 @@ RrColor *RrColorNew(const RrInstance *inst, gint r, gint g, gint b)
             out->pixel = xcol.pixel;
             out->key = key;
             out->refcount = 1;
-            g_hash_table_replace(RrColorHash(inst), &out->key, out);
+            g_hash_table_insert(RrColorHash(inst), &out->key, out);
         }
     }
     return out;
 }
 
-/*XXX same color could be pointed to twice, this might have to be a refcount*/
-
 void RrColorFree(RrColor *c)
 {
     if (c) {
         if (--c->refcount < 1) {
+            g_assert(g_hash_table_lookup(RrColorHash(c->inst), &c->key));
             g_hash_table_remove(RrColorHash(c->inst), &c->key);
             if (c->pixel) XFreeColors(RrDisplay(c->inst), RrColormap(c->inst),
                                       &c->pixel, 1, 0);
@@ -274,8 +292,7 @@ gulong RrColorPixel(const RrColor *c)
     return c->pixel;
 }
 
-GC RrColorGC(RrColor *c) /* XXX make this const RrColor* when the GCs are in
-                            a cache.. if possible? */
+GC RrColorGC(RrColor *c)
 {
     if (!c->gc)
         RrColorAllocateGC(c);