--- rxvt-unicode/src/rxvtimg.C	2012/06/08 17:22:48	1.62
+++ rxvt-unicode/src/rxvtimg.C	2012/06/09 11:15:29	1.69
@@ -211,7 +211,6 @@
   if (ref->cnt == 1 && ref->ours)
     return;
 
-  //TODO: maybe should reify instead
   Pixmap pm2 = XCreatePixmap (s->display->dpy, s->display->root, ref->w, ref->h, format->depth);
   GC gc = XCreateGC (s->display->dpy, pm, 0, 0);
   XCopyArea (s->display->dpy, pm, pm2, gc, 0, 0, ref->w, ref->h, 0, 0);
@@ -332,31 +331,70 @@
   return mask;
 }
 
+static void
+extract (int32_t cl0, int32_t cl1, int32_t &c, unsigned short &xc)
+{
+  int32_t x = clamp (c, cl0, cl1);
+  c -= x;
+  xc = x;
+}
+
+static bool
+extract (int32_t cl0, int32_t cl1, int32_t &r, int32_t &g, int32_t &b, int32_t &a, unsigned short &xr, unsigned short &xg, unsigned short &xb, unsigned short &xa)
+{
+  extract (cl0, cl1, r, xr);
+  extract (cl0, cl1, g, xg);
+  extract (cl0, cl1, b, xb);
+  extract (cl0, cl1, a, xa);
+
+  return xr | xg | xb | xa;
+}
+
 void
-rxvt_img::brightness (unsigned short r, unsigned short g, unsigned short b, unsigned short a)
+rxvt_img::brightness (int32_t r, int32_t g, int32_t b, int32_t a)
 {
+  unshare ();
+
   Display *dpy = s->display->dpy;
-  Picture src = create_xrender_mask (dpy, pm, True);
   Picture dst = XRenderCreatePicture (dpy, pm, format, 0, 0);
 
-  XRenderColor mask_c;
-  mask_c.red   = r;
-  mask_c.green = g;
-  mask_c.blue  = b;
-  mask_c.alpha = a;
-  XRenderFillRectangle (dpy, PictOpSrc, src, &mask_c, 0, 0, 1, 1);
+  // loop should not be needed for brightness, as only -1..1 makes sense
+
+  while (r | g | b | a)
+    {
+      unsigned short xr, xg, xb, xa;
+      XRenderColor mask_c;
+
+      if (extract (0, 65535, r, g, b, a, mask_c.red, mask_c.green, mask_c.blue, mask_c.alpha))
+        XRenderFillRectangle (dpy, PictOpAdd, dst, &mask_c, 0, 0, w, h);
+
+      if (extract (-65535, 0, r, g, b, a, mask_c.red, mask_c.green, mask_c.blue, mask_c.alpha))
+        {
+          XRenderColor mask_w = { 65535, 65535, 65535, 65535 };
+          XRenderFillRectangle (dpy, PictOpDifference, dst, &mask_w, 0, 0, w, h);
+          mask_c.red   = -mask_c.red;
+          mask_c.green = -mask_c.green;
+          mask_c.blue  = -mask_c.blue;
+          mask_c.alpha = -mask_c.alpha;
+          XRenderFillRectangle (dpy, PictOpAdd, dst, &mask_c, 0, 0, w, h);
+          XRenderFillRectangle (dpy, PictOpDifference, dst, &mask_w, 0, 0, w, h);
+        }
+    }
 
-  XRenderComposite (dpy, PictOpAdd, src, None, dst, 0, 0, 0, 0, 0, 0, w, h);
 
-  XRenderFreePicture (dpy, src);
   XRenderFreePicture (dpy, dst);
 }
 
 void
-rxvt_img::contrast (unsigned short r, unsigned short g, unsigned short b, unsigned short a)
+rxvt_img::contrast (int32_t r, int32_t g, int32_t b, int32_t a)
 {
   if (!(s->display->flags & DISPLAY_HAS_RENDER_MUL))
-    return;
+    {
+      rxvt_warn ("rxvt_img::contrast operation not supported on this display, RENDER extension too old.\n");
+      return;
+    }
+
+  unshare ();
 
   Display *dpy = s->display->dpy;
   Picture src = create_xrender_mask (dpy, pm, True);
@@ -472,10 +510,8 @@
     for (int j = 0; j < 3; ++j)
       xfrm.matrix [i][j] = XDoubleToFixed (matrix [i * 3 + j]);
 
-#if 0
   xfrm.matrix [0][2] -= XDoubleToFixed (x);//TODO
   xfrm.matrix [1][2] -= XDoubleToFixed (y);
-#endif
 
   XRenderSetPictureFilter (dpy, src, "good", 0, 0);
   XRenderSetPictureTransform (dpy, src, &xfrm);