X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=otk%2Frendercolor.cc;h=5da218ea92f84f101620240714e270ab89be83e8;hb=bfea000a7407e51b5659590415e410a47f6f046b;hp=3e20be1a06a56972f376563722025b42a50f3cc2;hpb=68482ff556b5b8b48bb536a4a95c5a1de01895cb;p=chaz%2Fopenbox

diff --git a/otk/rendercolor.cc b/otk/rendercolor.cc
index 3e20be1a..5da218ea 100644
--- a/otk/rendercolor.cc
+++ b/otk/rendercolor.cc
@@ -5,6 +5,7 @@
 #include "rendercolor.hh"
 #include "display.hh"
 #include "screeninfo.hh"
+#include "rendercontrol.hh"
 
 #include <cstdio>
 
@@ -27,21 +28,21 @@ RenderColor::RenderColor(int screen, unsigned char red,
   : _screen(screen),
     _red(red),
     _green(green),
-    _blue(blue),
-    _allocated(false)
+    _blue(blue)
 {
+  create();
 }
 
 RenderColor::RenderColor(int screen, RGB rgb)
   : _screen(screen),
     _red(rgb.r),
     _green(rgb.g),
-    _blue(rgb.b),
-    _allocated(false)
+    _blue(rgb.b)
 {
+  create();
 }
 
-void RenderColor::create() const
+void RenderColor::create()
 {
   unsigned long color = _blue | _green << 8 | _red << 16;
   
@@ -62,13 +63,8 @@ void RenderColor::create() const
     xcol.red = (_red << 8) | _red;
     xcol.green = (_green << 8) | _green;
     xcol.blue = (_blue << 8) | _blue;
-    xcol.pixel = 0;
 
-    if (! XAllocColor(**display, info->colormap(), &xcol)) {
-      fprintf(stderr, "RenderColor: color alloc error: rgb:%x/%x/%x\n",
-	      _red, _green, _blue);
-      xcol.pixel = 0;
-    }
+    display->renderControl(_screen)->allocateColor(&xcol);
 
     _pixel = xcol.pixel;
     gcv.foreground = _pixel;
@@ -82,39 +78,23 @@ void RenderColor::create() const
     _cache[_screen][color] = item;
     ++item->count;
   }
-
-  _allocated = true;
-}
-
-unsigned long RenderColor::pixel() const
-{
-  if (!_allocated) create();
-  return _pixel;
-}
-
-GC RenderColor::gc() const
-{
-  if (!_allocated) create();
-  return _gc;
 }
 
 RenderColor::~RenderColor()
 {
   unsigned long color = _blue | _green << 8 | _red << 16;
 
-  if (_allocated) {
-    CacheItem *item = _cache[_screen][color];
-    assert(item); // better be...
-
-    if (--item->count <= 0) {
-      // remove from the cache
-      XFreeGC(**display, _gc);
-      _cache[_screen][color] = 0;
-      delete item;
+  CacheItem *item = _cache[_screen][color];
+  assert(item); // better be...
+  
+  if (--item->count <= 0) {
+    // remove from the cache
+    XFreeGC(**display, _gc);
+    _cache[_screen][color] = 0;
+    delete item;
 
-      const ScreenInfo *info = display->screenInfo(_screen);
-      XFreeColors(**display, info->colormap(), &_pixel, 1, 0);
-    }
+    const ScreenInfo *info = display->screenInfo(_screen);
+    XFreeColors(**display, info->colormap(), &_pixel, 1, 0);
   }
 }