all repos — openbox @ 3fe7bded58fb82fd2c1c2a5f9ad2e8f453fa0609

openbox fork - make it a bit more like ryudo

increment the count 1 right off the start for the cache
Dana Jansens danakj@orodu.net
commit

3fe7bded58fb82fd2c1c2a5f9ad2e8f453fa0609

parent

77882b46dbeb1f626f052e2eb3cb200d44f3e3ff

1 files changed, 8 insertions(+), 2 deletions(-)

jump to
M otk/rendercolor.ccotk/rendercolor.cc

@@ -47,6 +47,8 @@ void RenderColor::create()

{ unsigned long color = _blue | _green << 8 | _red << 16; + printf("CREATE COLOR: %lx\n", color); + // try get a gc from the cache CacheItem *item = _cache[_screen][color];

@@ -79,17 +81,21 @@ GCForeground | GCCapStyle, &gcv);

assert(_gc); // insert into the cache - _cache[_screen][color] = new CacheItem(_gc); + item = new CacheItem(_gc); + _cache[_screen][color] = item; + ++item->count; } } RenderColor::~RenderColor() { unsigned long color = _blue | _green << 8 | _red << 16; - + CacheItem *item = _cache[_screen][color]; assert(item); // it better be in the cache ... + printf("DESTROY COLOR: %lx %d\n", color, item->count); + if (--item->count <= 0) { // remove from the cache XFreeGC(**display, _gc);