[ViewVC] Diff of: cvs/rxvt-unicode/src/rxvttoolkit.C

Comparing rxvt-unicode/src/rxvttoolkit.C (file contents):
Revision 1.143 by sf-exg, Sat Nov 1 10:17:55 2014 UTC vs.
Revision 1.149 by root, Wed Dec 17 16:03:51 2014 UTC

 #include "../config.h"
 #include <rxvt.h>
 #include <rxvttoolkit.h>
+#include <stdlib.h>
 #include <unistd.h>
 #include <fcntl.h>
 #include <sys/utsname.h>
 #include <sys/socket.h>
   flags = 0;
 #if XRENDER
   int major, minor;
   if (XRenderQueryVersion (dpy, &major, &minor))
-    if (major > 0 || (major == 0 && minor >= 10))
+    if (major > 0 || (major == 0 && minor >= 11))
       {
         flags |= DISPLAY_HAS_RENDER;
-#if 0
-        if (major > 0 || (major == 0 && minor >= 11))
-          flags |= DISPLAY_HAS_RENDER_MUL;
-#endif
         if (XFilters *filters = XRenderQueryFilters (dpy, root))
           {
             for (int i = 0; i < filters->nfilter; i++)
               if (!strcmp (filters->filter [i], FilterConvolution))
   unsigned int alpha = color.a >= 0xff00 ? 0xffff : color.a;
 #if XFT
   XRenderPictFormat *format;
+  // not needed by XftColorAlloc, but by the other paths (ours
+  // and fallback), so just set all components here.
+  c.color.red   = color.r;
+  c.color.green = color.g;
+  c.color.blue  = color.b;
+  c.color.alpha = alpha;
   // FUCKING Xft gets it wrong, of course, so work around it.
   // Transparency users should eat shit and die, and then
   // XRenderQueryPictIndexValues themselves plenty.
   if ((screen->visual->c_class == TrueColor)
       && (format = XRenderFindVisualFormat (screen->dpy, screen->visual)))
     {
       // the fun lies in doing everything manually...
-      c.color.red   = color.r;
-      c.color.green = color.g;
-      c.color.blue  = color.b;
-      c.color.alpha = alpha;
       // Xft wants premultiplied alpha, but abuses the alpha channel
       // as blend factor, and doesn't allow us to set the alpha channel
       c.color.red   = c.color.red   * alpha / 0xffff;
       c.color.green = c.color.green * alpha / 0xffff;
       d.red   = color.r;
       d.green = color.g;
       d.blue  = color.b;
       d.alpha = alpha;
+      // XftColorAlloc always returns 100% transparent pixels(!)
       if (XftColorAllocValue (screen->dpy, screen->visual, screen->cmap, &d, &c))
         return true;
     }
 #else
   c.red   = color.r;
     }
   else if (XAllocColor (screen->dpy, screen->cmap, &c))
     return true;
 #endif
-  c.pixel = (color.r + color.g + color.b) > 128*3
+  c.pixel = (color.r * 2 + color.g * 3 + color.b) >= 0x8000 * 6
           ? WhitePixelOfScreen (DefaultScreenOfDisplay (screen->dpy))
           : BlackPixelOfScreen (DefaultScreenOfDisplay (screen->dpy));
   return false;
 }
       // many kilobytes transfer per colour, but pseudocolor isn't worth
       // many extra optimisations.
       XQueryColors (screen->dpy, screen->cmap, colors, cmap_size);
+      while (cmap_size)
+        {
-      int diff = 0x7fffffffUL;
+          int diff = 0x7fffffffL;
-      XColor *best = colors;
+          XColor *best = colors;
-      for (int i = 0; i < cmap_size; i++)
+          for (int i = 0; i < cmap_size; i++)
-        {
-          int d = (squared_diff<int> (color.r >> 2, colors [i].red   >> 2))
-                + (squared_diff<int> (color.g >> 2, colors [i].green >> 2))
-                + (squared_diff<int> (color.b >> 2, colors [i].blue  >> 2));
-          if (d < diff)
             {
+              // simple weighted rgb distance sucks, but keeps it simple
+              int d = abs (color.r - colors [i].red  ) * 2
+                    + abs (color.g - colors [i].green) * 3
+                    + abs (color.b - colors [i].blue );
+              if (d < diff)
+                {
-              diff = d;
+                  diff = d;
-              best = colors + i;
+                  best = colors + i;
+                }
             }
-        }
-      //rxvt_warn ("could not allocate %04x %04x %04x, getting %04x %04x %04x instead (%d)\n",
+          //rxvt_warn ("could not allocate %04x %04x %04x, getting %04x %04x %04x instead (%d,%d)\n",
-      //    color.r, color.g, color.b, best->red, best->green, best->blue, diff);
+          //    color.r, color.g, color.b, best->red, best->green, best->blue, diff, best - colors);
-      got = alloc (screen, rgba (best->red, best->green, best->blue));
+          got = alloc (screen, rgba (best->red, best->green, best->blue));
+          if (got)
+            break;
+          *best = colors [--cmap_size];
+        }
       delete [] colors;
     }
 #endif
 void
 rxvt_color::get (rgba &color) const
 {
 #if XFT
   color.r = c.color.red;
   color.g = c.color.green;
   color.b = c.color.blue;
   color.a = c.color.alpha;
+  if (IN_RANGE_INC (color.a, 0x0001, 0xfffe))
+    {
+      color.r = color.r * 0xffff / color.a;
+      color.g = color.g * 0xffff / color.a;
+      color.b = color.b * 0xffff / color.a;
+    }
 #else
   color.r = c.red;
   color.g = c.green;
   color.b = c.blue;
   color.a = rgba::MAX_CC;
 #endif
 }
 void
 rxvt_color::get (XColor &color) const

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing rxvt-unicode/src/rxvttoolkit.C (file contents): Revision 1.143 by sf-exg, Sat Nov 1 10:17:55 2014 UTC vs. Revision 1.149 by root, Wed Dec 17 16:03:51 2014 UTC

Diff Legend

Comparing rxvt-unicode/src/rxvttoolkit.C (file contents):
Revision 1.143 by sf-exg, Sat Nov 1 10:17:55 2014 UTC vs.
Revision 1.149 by root, Wed Dec 17 16:03:51 2014 UTC