diff --git a/CHANGES b/CHANGES index 1e308cc90..b985130c2 100644 --- a/CHANGES +++ b/CHANGES @@ -6,6 +6,8 @@ CHANGES IN FLTK 1.1.8 STR #1742, STR #1777, STR #1794, STR #1827, STR #1843, STR #1796, STR #1815, STR #1726, STR #1753, STR #1855, STR #1862, STR #1867, STR #1874) + - Fixed image read for partial regions on X11 + (STR #1716) - Fixed KDE/Gnome icon paths (STR #1795) - Fixed Tab key to wrap around menu bars (STR #1877) - Fixed possible timer leak in Scrollbar (STR #1880) diff --git a/src/fl_read_image.cxx b/src/fl_read_image.cxx index f0370a9c6..e2d6b61e2 100644 --- a/src/fl_read_image.cxx +++ b/src/fl_read_image.cxx @@ -26,6 +26,7 @@ // #include +#include #include #include "flstring.h" @@ -41,12 +42,42 @@ # include # ifdef __sgi # include +# else +# include # endif // __sgi // Defined in fl_color.cxx extern uchar fl_redmask, fl_greenmask, fl_bluemask; extern int fl_redshift, fl_greenshift, fl_blueshift, fl_extrashift; +// +// 'fl_subimage_offsets()' - Calculate subimage offsets for an axis +static inline int +fl_subimage_offsets(int a, int aw, int b, int bw, int &obw) +{ + int off; + int ob; + + if (b >= a) { + ob = b; + off = 0; + } else { + ob = a; + off = a - b; + } + + bw -= off; + + if (ob + bw <= a + aw) { + obw = bw; + } else { + obw = (a + aw) - ob; + } + + return off; +} + + // // 'fl_read_image()' - Read an image from the current window. // @@ -92,7 +123,42 @@ fl_read_image(uchar *p, // I - Pixel buffer or NULL to allocate # endif // __sgi if (!image) { + // fetch absolute coordinates + int dx, dy; + Window child_win; + XTranslateCoordinates(fl_display, fl_window, + RootWindow(fl_display, fl_screen), X, Y, &dx, &dy, &child_win); + + // screen dimensions + int sx, sy, sw, sh; + Fl::screen_xywh(sx, sy, sw, sh, fl_screen); + + if (dx >= sx && dy >= sy && dx + w <= sw && dy + h <= sh) { + // the image is fully contained, we can use the traditional method image = XGetImage(fl_display, fl_window, X, Y, w, h, AllPlanes, ZPixmap); + } else { + // image is crossing borders, determine visible region + int nw, nh, noffx, noffy; + noffx = fl_subimage_offsets(sx, sw, dx, w, nw); + noffy = fl_subimage_offsets(sy, sh, dy, h, nh); + if (nw <= 0 || nh <= 0) return 0; + + // allocate the image + int bpp = fl_visual->depth + ((fl_visual->depth / 8) % 2) * 8; + char* buf = (char*)malloc(bpp / 8 * w * h); + image = XCreateImage(fl_display, fl_visual->visual, + fl_visual->depth, ZPixmap, 0, buf, w, h, bpp, 0); + if (!image) { + if (buf) free(buf); + return 0; + } + + if (!XGetSubImage(fl_display, fl_window, X + noffx, Y + noffy, + nw, nh, AllPlanes, ZPixmap, image, noffx, noffy)) { + XDestroyImage(image); + return 0; + } + } } if (!image) return 0;