From 4755ace9e9252c7cac6e29143fd0662ae62c2bf4 Mon Sep 17 00:00:00 2001
From: Manolo Gouy <Manolo>
Date: Mon, 28 Nov 2016 11:32:59 +0000
Subject: X11 platform: fix STR#3353 and correct drawing of depth-2 RGB images
 when XRender extension is present.

Fl_RGB_Image's of depth-2, that is, monocolor and transparent, were not drawn using the XRender extension,
when present, as are depth-4 images. With this change they are.
A by-product of this change is that it fixes an erroneous drawing (STR#3353) that occurred when
the image expands outside the window.

git-svn-id: file:///fltk/svn/fltk/branches/branch-1.4@12124 ea41ed52-d2ee-0310-a9c1-e6b18d33e121
---
 src/drivers/Xlib/Fl_Xlib_Graphics_Driver_image.cxx | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_image.cxx b/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_image.cxx
index 66a25ed01..543e2ba7f 100644
--- a/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_image.cxx
+++ b/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_image.cxx
@@ -33,7 +33,7 @@
 
 // scanline_pad must be a power of 2 and greater or equal to 8.
 
-// PsuedoColor visuals must have 8 bits_per_pixel (although the depth
+// PseudoColor visuals must have 8 bits_per_pixel (although the depth
 // may be less than 8).  This is the only limitation that affects any
 // modern X displays, you can't use 12 or 16 bit colormaps.
 
@@ -321,6 +321,13 @@ static void argb_premul_converter(const uchar *from, uchar *to, int w, int delta
              ((from[2] * from[3]) / 255));
 }
 
+static void depth2_to_argb_premul_converter(const uchar *from, uchar *to, int w, int delta) {
+  INNARDS32((unsigned(from[1]) << 24) +
+            (((from[0] * from[1]) / 255) << 16) +
+            (((from[0] * from[1]) / 255) << 8) +
+            ((from[0] * from[1]) / 255));
+}
+
 static void bgrx_converter(const uchar *from, uchar *to, int w, int delta) {
   INNARDS32((from[0]<<8)+(from[1]<<16)+(unsigned(from[2])<<24));
 }
@@ -478,7 +485,7 @@ static void innards(const uchar *buf, int X, int Y, int W, int H,
   if (alpha) {
     // This flag states the destination format is ARGB32 (big-endian), pre-multiplied.
     bytes_per_pixel = 4;
-    conv = argb_premul_converter;
+    conv = (mono ? depth2_to_argb_premul_converter : argb_premul_converter);
     xi.depth = 32;
     xi.bits_per_pixel = 32;
 
@@ -701,7 +708,7 @@ void Fl_Xlib_Graphics_Driver::draw(Fl_RGB_Image *img, int XP, int YP, int WP, in
     int depth = img->d();
     if (depth == 1 || depth == 3) {
       surface = new Fl_Image_Surface(img->w(), img->h());
-    } else if (depth == 4 && can_do_alpha_blending()) {
+    } else if (can_do_alpha_blending()) {
       Fl_Offscreen pixmap = XCreatePixmap(fl_display, RootWindow(fl_display, fl_screen), img->w(), img->h(), 32);
       surface = new Fl_Image_Surface(img->w(), img->h(), 0, pixmap);
       depth |= FL_IMAGE_WITH_ALPHA;
@@ -715,7 +722,7 @@ void Fl_Xlib_Graphics_Driver::draw(Fl_RGB_Image *img, int XP, int YP, int WP, in
     }
   }
   if (*Fl_Graphics_Driver::id(img)) {
-    if (img->d() == 4)
+    if (img->d() == 4 || img->d() == 2)
       copy_offscreen_with_alpha(X, Y, W, H, *Fl_Graphics_Driver::id(img), cx, cy);
     else
       copy_offscreen(X, Y, W, H, *Fl_Graphics_Driver::id(img), cx, cy);
-- 
cgit v1.2.3