X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=otk%2Fpseudorendercontrol.cc;h=cf9929248586e4e2c12d0b002eba771bcfb610fe;hb=8269fc2b3965d12ba308caa554bfa7ee037fba13;hp=29636ee9ac21fc4b7e5e4e91a971890ddfb9279c;hpb=01a37dfe1888f1ac6da7ab12c6e4d59ce104d12c;p=chaz%2Fopenbox

diff --git a/otk/pseudorendercontrol.cc b/otk/pseudorendercontrol.cc
index 29636ee9..cf992924 100644
--- a/otk/pseudorendercontrol.cc
+++ b/otk/pseudorendercontrol.cc
@@ -9,14 +9,12 @@
 #include "rendertexture.hh"
 
 extern "C" {
-#ifdef    HAVE_STDLIB_H
-#  include <stdlib.h>
-#endif // HAVE_STDLIB_H
-
 #include "../src/gettext.h"
 #define _(str) gettext(str)
 }
 
+#include <cstdlib>
+
 namespace otk {
 
 PseudoRenderControl::PseudoRenderControl(int screen)
@@ -113,65 +111,47 @@ PseudoRenderControl::~PseudoRenderControl()
   delete [] _colors;
 }
 
+inline const XColor *PseudoRenderControl::pickColor(int r, int g, int b) const
+{
+  r = (r & 0xff) >> (8-_bpc);
+  g = (g & 0xff) >> (8-_bpc);
+  b = (b & 0xff) >> (8-_bpc);
+  return &_colors[(r << (2*_bpc)) + (g << (1*_bpc)) + b];
+}
+
 void PseudoRenderControl::reduceDepth(Surface &sf, XImage *im) const
 {
   pixel32 *data = sf.pixelData();
-  char *p = (char *)data;
-  int x, y, r, g, b;
-    for (y = 0; y < im->height; y++) {
-      for (x = 0; x < im->width; x++) {
-        r = (data[x] >> default_red_shift) & 0xFF;
-        r = r >> (8-_bpc);
-        g = (data[x] >> default_green_shift) & 0xFF;
-        g = g >> (8-_bpc);
-        b = (data[x] >> default_blue_shift) & 0xFF;
-        b = b >> (8-_bpc);
-        p[x] = _colors[(r << (2*_bpc)) + (g << (1*_bpc)) + b].pixel;
-      }
-      data += im->width;
-      p += im->bytes_per_line;
+  pixel32 *ret = (pixel32*)malloc(im->width * im->height * 4);
+  char *p = (char *)ret;
+  int x, y;
+  for (y = 0; y < im->height; y++) {
+    for (x = 0; x < im->width; x++) {
+      p[x] = pickColor(data[x] >> default_red_shift,
+                       data[x] >> default_green_shift,
+                       data[x] >> default_blue_shift)->pixel;
     }
-
+    data += im->width;
+    p += im->bytes_per_line;
+  }
+  im->data = (char*)ret;
 }
 
 void PseudoRenderControl::allocateColor(XColor *color) const
 {
-  const ScreenInfo *info = display->screenInfo(_screen);
-  int depth = info->depth();
-
-  // get the allocated values from the X server (only the first 256 XXX why!?)
-  XColor icolors[256];
-  int incolors = (((1 << depth) > 256) ? 256 : (1 << depth));
-  for (int i = 0; i < incolors; i++)
-    icolors[i].pixel = i;
-  XQueryColors(**display, info->colormap(), icolors, incolors);
+  const XColor *c = pickColor(color->red, color->blue, color->green);
 
-  unsigned long closest = 0xffffffff, close = 0;
-  for (int ii = 0; ii < incolors; ii++) {
-    // find deviations
-    int r = (color->red - icolors[ii].red) & 0xff;
-    int g = (color->green - icolors[ii].green) & 0xff;
-    int b = (color->blue - icolors[ii].blue) & 0xff;
-    // find a weighted absolute deviation
-    unsigned long dev = (r * r) + (g * g) + (b * b);
-
-    if (dev < closest) {
-      closest = dev;
-      close = ii;
-    }
-  }
-
-  color->red = icolors[close].red;
-  color->green = icolors[close].green;
-  color->blue = icolors[close].blue;
-  color->pixel = icolors[close].pixel;
+  color->red = c->red;
+  color->green = c->green;
+  color->blue = c->blue;
+  color->pixel = c->pixel;
 
-  // try alloc this closest color, it had better succeed!
-  if (XAllocColor(**display, info->colormap(), color)) {
+  if (XAllocColor(**display, display->screenInfo(_screen)->colormap(), color))
     color->flags = DoRed|DoGreen|DoBlue; // mark as alloced
-  }
   else
     assert(false); // wtf has gone wrong, its already alloced for chissake!
+
+  return;
 }
 
 }