--- rxvt-unicode/src/rxvtimg.C 2012/06/07 20:22:40 1.53 +++ rxvt-unicode/src/rxvtimg.C 2012/06/07 20:32:58 1.57 @@ -73,15 +73,17 @@ // that both 24 and 32 bpp MUST be supported by any screen that supports xrender int depth = gdk_pixbuf_get_has_alpha (pb) ? 32 : 24; + int byte_order = ecb_big_endian () ? MSBFirst : LSBFirst; + XImage xi; xi.width = width; xi.height = height; xi.xoffset = 0; xi.format = ZPixmap; - xi.byte_order = LSBFirst; // maybe go for host byte order, because servers are usually local? - xi.bitmap_unit = 32; //XY only, unused - xi.bitmap_bit_order = LSBFirst; //XY only, unused + xi.byte_order = ImageByteOrder (dpy); + xi.bitmap_unit = 0; //XY only, unused + xi.bitmap_bit_order = 0; //XY only, unused xi.bitmap_pad = BitmapPad (dpy); xi.depth = depth; xi.bytes_per_line = 0; @@ -91,6 +93,8 @@ xi.blue_mask = 0x00000000; //Z only, unused xi.obdata = 0; // probably unused + bool byte_order_mismatch = byte_order != xi.byte_order; + if (!XInitImage (&xi)) rxvt_fatal ("unable to initialise ximage, please report.\n"); @@ -119,7 +123,7 @@ uint32_t v = (r << 16) | (g << 8) | b; - if (ecb_big_endian ()) + if (ecb_big_endian () ? !byte_order_mismatch : byte_order_mismatch) v = ecb_bswap32 (v); *dst++ = v; @@ -129,10 +133,13 @@ { uint32_t v = *(uint32_t *)src; src += 4; - if (ecb_little_endian ()) + if (ecb_big_endian ()) v = ecb_bswap32 (v); - v = ecb_rotr32 (v, 8); + v = ecb_rotl32 (v, 8); + + if (!byte_order_mismatch) + v = ecb_bswap32 (v); *dst++ = v; }