_red(red),
_green(green),
_blue(blue),
- _gc(0)
+ _allocated(false),
+ _created(false)
{
- create();
}
RenderColor::RenderColor(int screen, RGB rgb)
_red(rgb.r),
_green(rgb.g),
_blue(rgb.b),
- _gc(0)
+ _allocated(false),
+ _created(false)
{
- create();
}
-void RenderColor::create()
+void RenderColor::create() const
{
unsigned long color = _blue | _green << 8 | _red << 16;
xcol.blue = (_blue << 8) | _blue;
xcol.pixel = 0;
- if (! XAllocColor(**display, info->colormap(), &xcol)) {
+ if (!XAllocColor(**display, info->colormap(), &xcol)) {
fprintf(stderr, "RenderColor: color alloc error: rgb:%x/%x/%x\n",
_red, _green, _blue);
xcol.pixel = 0;
- }
+ } else
+ _allocated = true;
_pixel = xcol.pixel;
gcv.foreground = _pixel;
_cache[_screen][color] = item;
++item->count;
}
+
+ _created = true;
+}
+
+unsigned long RenderColor::pixel() const
+{
+ if (!_created) create();
+ return _pixel;
+}
+
+GC RenderColor::gc() const
+{
+ if (!_created) create();
+ return _gc;
}
RenderColor::~RenderColor()
{
unsigned long color = _blue | _green << 8 | _red << 16;
- CacheItem *item = _cache[_screen][color];
- assert(item); // it better be in the cache ...
+ if (_created) {
+ 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;
+ if (--item->count <= 0) {
+ // remove from the cache
+ XFreeGC(**display, _gc);
+ _cache[_screen][color] = 0;
+ delete item;
+
+ if (_allocated) {
+ const ScreenInfo *info = display->screenInfo(_screen);
+ XFreeColors(**display, info->colormap(), &_pixel, 1, 0);
+ }
+ }
}
}