/[rdesktop]/jpeg/rdesktop/trunk/xwin.c

This is repository of my old source code which isn't updated any more. Go to git.rot13.org for current projects!

Diff of /jpeg/rdesktop/trunk/xwin.c

Parent Directory | Revision Log | View Patch Patch

-sourceforge.net/trunk/rdesktop/xwin.c
revision 498 by astrand,
Tue Oct 14 12:05:27 2003 UTC
+jpeg/rdesktop/trunk/xwin.c
revision 1508 by dpavlin,
Mon Jul 20 16:47:49 2009 UTC
 Line 1
  /* -*- c-basic-offset: 8 -*-
     rdesktop: A Remote Desktop Protocol client.
     User interface services - X Window System
-    Copyright (C) Matthew Chapman 1999-2002
+    Copyright (C) Matthew Chapman 1999-2008
+    Copyright 2007-2008 Pierre Ossman <ossman@cendio.se> for Cendio AB
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
-Line 20
+Line 21
  #include <X11/Xlib.h>
  #include <X11/Xutil.h>
+ #include <X11/Xproto.h>
+ #include <unistd.h>
+ #include <sys/time.h>
  #include <time.h>
  #include <errno.h>
+ #include <strings.h>
  #include "rdesktop.h"
  #include "xproto.h"
+ /* DJ globals begin */
+ extern uint8 * g_bitmap_data;
+ extern uint8 * g_bitmap_data_last_write;
+ extern uint32 g_pixels_changed;
+ extern unsigned long long g_time_last_change;
+ /* DJ globals end */
  extern int g_width;
  extern int g_height;
- extern BOOL g_sendmotion;
+ extern int g_xpos;
- extern BOOL g_fullscreen;
+ extern int g_ypos;
- extern BOOL g_grab_keyboard;
+ extern int g_pos;
- extern BOOL g_hide_decorations;
+ extern RD_BOOL g_sendmotion;
+ extern RD_BOOL g_fullscreen;
+ extern RD_BOOL g_grab_keyboard;
+ extern RD_BOOL g_hide_decorations;
  extern char g_title[];
- extern int g_server_bpp;
+ /* Color depth of the RDP session.
+    As of RDP 5.1, it may be 8, 15, 16 or 24. */
+ extern int g_server_depth;
  extern int g_win_button_size;
  Display *g_display;
-Line 40 
 Time g_last_gesturetime;
+Line 57 
 Time g_last_gesturetime;
  static int g_x_socket;
  static Screen *g_screen;
  Window g_wnd;
- BOOL g_enable_compose = False;
- static GC g_gc;
+ /* SeamlessRDP support */
+ typedef struct _seamless_group
+ {
+         Window wnd;
+         unsigned long id;
+         unsigned int refcnt;
+ } seamless_group;
+ typedef struct _seamless_window
+ {
+         Window wnd;
+         unsigned long id;
+         unsigned long behind;
+         seamless_group *group;
+         int xoffset, yoffset;
+         int width, height;
+         int state;              /* normal/minimized/maximized. */
+         unsigned int desktop;
+         struct timeval *position_timer;
+         RD_BOOL outstanding_position;
+         unsigned int outpos_serial;
+         int outpos_xoffset, outpos_yoffset;
+         int outpos_width, outpos_height;
+         unsigned int icon_size;
+         unsigned int icon_offset;
+         char icon_buffer[32 * 32 * 4];
+         struct _seamless_window *next;
+ } seamless_window;
+ static seamless_window *g_seamless_windows = NULL;
+ static unsigned long g_seamless_focused = 0;
+ static RD_BOOL g_seamless_started = False;      /* Server end is up and running */
+ static RD_BOOL g_seamless_active = False;       /* We are currently in seamless mode */
+ static RD_BOOL g_seamless_hidden = False;       /* Desktop is hidden on server */
+ static RD_BOOL g_seamless_broken_restack = False;       /* WM does not properly restack */
+ extern RD_BOOL g_seamless_rdp;
+ extern uint32 g_embed_wnd;
+ RD_BOOL g_enable_compose = False;
+ RD_BOOL g_Unobscured;           /* used for screenblt */
+ static GC g_gc = NULL;
+ static GC g_create_bitmap_gc = NULL;
+ static GC g_create_glyph_gc = NULL;
+ static XRectangle g_clip_rectangle;
  static Visual *g_visual;
+ /* Color depth of the X11 visual of our window (e.g. 24 for True Color R8G8B visual).
+    This may be 32 for R8G8B8 visuals, and then the rest of the bits are undefined
+    as far as we're concerned. */
  static int g_depth;
+ /* Bits-per-Pixel of the pixmaps we'll be using to draw on our window.
+    This may be larger than g_depth, in which case some of the bits would
+    be kept solely for alignment (e.g. 32bpp pixmaps on a 24bpp visual). */
  static int g_bpp;
  static XIM g_IM;
  static XIC g_IC;
  static XModifierKeymap *g_mod_map;
+ /* Maps logical (xmodmap -pp) pointing device buttons (0-based) back
+    to physical (1-based) indices. */
+ static unsigned char g_pointer_log_to_phys_map[32];
  static Cursor g_current_cursor;
+ static RD_HCURSOR g_null_cursor = NULL;
  static Atom g_protocol_atom, g_kill_atom;
- static BOOL g_focused;
+ extern Atom g_net_wm_state_atom;
- static BOOL g_mouse_in_wnd;
+ extern Atom g_net_wm_desktop_atom;
+ static RD_BOOL g_focused;
+ static RD_BOOL g_mouse_in_wnd;
+ /* Indicates that:
+) visual has 15, 16 or 24 depth and the same color channel masks
+       as its RDP equivalent (implies X server is LE),
+) host is LE
+    This will trigger an optimization whose real value is questionable.
+ */
+ static RD_BOOL g_compatible_arch;
+ /* Indicates whether RDP's bitmaps and our XImages have the same
+    binary format. If so, we can avoid an expensive translation.
+    Note that this can be true when g_compatible_arch is false,
+    e.g.:
+      RDP(LE) <-> host(BE) <-> X-Server(LE)
+    ('host' is the machine running rdesktop; the host simply memcpy's
+     so its endianess doesn't matter)
+  */
+ static RD_BOOL g_no_translate_image = False;
  /* endianness */
- static BOOL g_host_be;
+ static RD_BOOL g_host_be;
- static BOOL g_xserver_be;
+ static RD_BOOL g_xserver_be;
+ static int g_red_shift_r, g_blue_shift_r, g_green_shift_r;
+ static int g_red_shift_l, g_blue_shift_l, g_green_shift_l;
  /* software backing store */
- static BOOL g_ownbackstore;
+ extern RD_BOOL g_ownbackstore;
- static Pixmap g_backstore;
+ static Pixmap g_backstore = 0;
  /* Moving in single app mode */
- static BOOL g_moving_wnd;
+ static RD_BOOL g_moving_wnd;
  static int g_move_x_offset = 0;
  static int g_move_y_offset = 0;
+ static RD_BOOL g_using_full_workarea = False;
  #ifdef WITH_RDPSND
- extern int g_dsp_fd;
+ extern RD_BOOL g_rdpsnd;
- extern BOOL g_dsp_busy;
  #endif
  /* MWM decorations */
-Line 76 
 extern BOOL g_dsp_busy;
+Line 169 
 extern BOOL g_dsp_busy;
  #define PROP_MOTIF_WM_HINTS_ELEMENTS    5
  typedef struct
  {
-         uint32 flags;
+         unsigned long flags;
-         uint32 functions;
+         unsigned long functions;
-         uint32 decorations;
+         unsigned long decorations;
-         sint32 inputMode;
+         long inputMode;
-         uint32 status;
+         unsigned long status;
  }
  PropMotifWmHints;
-Line 92 
 typedef struct
+Line 185 
 typedef struct
  }
  PixelColour;
+ #define ON_ALL_SEAMLESS_WINDOWS(func, args) \
+         do { \
+                 seamless_window *sw; \
+                 XRectangle rect; \
+                 if (!g_seamless_windows) break; \
+                 for (sw = g_seamless_windows; sw; sw = sw->next) { \
+                     rect.x = g_clip_rectangle.x - sw->xoffset; \
+                     rect.y = g_clip_rectangle.y - sw->yoffset; \
+                     rect.width = g_clip_rectangle.width; \
+                     rect.height = g_clip_rectangle.height; \
+                     XSetClipRectangles(g_display, g_gc, 0, 0, &rect, 1, YXBanded); \
+                     func args; \
+                 } \
+                 XSetClipRectangles(g_display, g_gc, 0, 0, &g_clip_rectangle, 1, YXBanded); \
+         } while (0)
+ static void
+ seamless_XFillPolygon(Drawable d, XPoint * points, int npoints, int xoffset, int yoffset)
+ {
+         points[0].x -= xoffset;
+         points[0].y -= yoffset;
+         XFillPolygon(g_display, d, g_gc, points, npoints, Complex, CoordModePrevious);
+         points[0].x += xoffset;
+         points[0].y += yoffset;
+ }
+ static void
+ seamless_XDrawLines(Drawable d, XPoint * points, int npoints, int xoffset, int yoffset)
+ {
+         points[0].x -= xoffset;
+         points[0].y -= yoffset;
+         XDrawLines(g_display, d, g_gc, points, npoints, CoordModePrevious);
+         points[0].x += xoffset;
+         points[0].y += yoffset;
+ }
  #define FILL_RECTANGLE(x,y,cx,cy)\
  { \
          XFillRectangle(g_display, g_wnd, g_gc, x, y, cx, cy); \
+         ON_ALL_SEAMLESS_WINDOWS(XFillRectangle, (g_display, sw->wnd, g_gc, x-sw->xoffset, y-sw->yoffset, cx, cy)); \
          if (g_ownbackstore) \
                  XFillRectangle(g_display, g_backstore, g_gc, x, y, cx, cy); \
  }
-Line 105 
 PixelColour;
+Line 234 
 PixelColour;
          XFillRectangle(g_display, g_ownbackstore ? g_backstore : g_wnd, g_gc, x, y, cx, cy); \
  }
+ #define FILL_POLYGON(p,np)\
+ { \
+         XFillPolygon(g_display, g_wnd, g_gc, p, np, Complex, CoordModePrevious); \
+         if (g_ownbackstore) \
+                 XFillPolygon(g_display, g_backstore, g_gc, p, np, Complex, CoordModePrevious); \
+         ON_ALL_SEAMLESS_WINDOWS(seamless_XFillPolygon, (sw->wnd, p, np, sw->xoffset, sw->yoffset)); \
+ }
+ #define DRAW_ELLIPSE(x,y,cx,cy,m)\
+ { \
+         switch (m) \
+         { \
+                 case 0: /* Outline */ \
+                         XDrawArc(g_display, g_wnd, g_gc, x, y, cx, cy, 0, 360*64); \
+                         ON_ALL_SEAMLESS_WINDOWS(XDrawArc, (g_display, sw->wnd, g_gc, x-sw->xoffset, y-sw->yoffset, cx, cy, 0, 360*64)); \
+                         if (g_ownbackstore) \
+                                 XDrawArc(g_display, g_backstore, g_gc, x, y, cx, cy, 0, 360*64); \
+                         break; \
+                 case 1: /* Filled */ \
+                         XFillArc(g_display, g_wnd, g_gc, x, y, cx, cy, 0, 360*64); \
+                         ON_ALL_SEAMLESS_WINDOWS(XFillArc, (g_display, sw->wnd, g_gc, x-sw->xoffset, y-sw->yoffset, cx, cy, 0, 360*64)); \
+                         if (g_ownbackstore) \
+                                 XFillArc(g_display, g_backstore, g_gc, x, y, cx, cy, 0, 360*64); \
+                         break; \
+         } \
+ }
  /* colour maps */
- BOOL g_owncolmap = False;
+ extern RD_BOOL g_owncolmap;
  static Colormap g_xcolmap;
  static uint32 *g_colmap = NULL;
- #define TRANSLATE(col)          ( g_server_bpp != 8 ? translate_colour(col) : g_owncolmap ? col : translate_colour(g_colmap[col]) )
+ #define TRANSLATE(col)          ( g_server_depth != 8 ? translate_colour(col) : g_owncolmap ? col : g_colmap[col] )
  #define SET_FOREGROUND(col)     XSetForeground(g_display, g_gc, TRANSLATE(col));
  #define SET_BACKGROUND(col)     XSetBackground(g_display, g_gc, TRANSLATE(col));
-Line 136 
 static int rop2_map[] = {
+Line 292 
 static int rop2_map[] = {
  #define SET_FUNCTION(rop2)      { if (rop2 != ROP2_COPY) XSetFunction(g_display, g_gc, rop2_map[rop2]); }
  #define RESET_FUNCTION(rop2)    { if (rop2 != ROP2_COPY) XSetFunction(g_display, g_gc, GXcopy); }
+ static seamless_window *
+ sw_get_window_by_id(unsigned long id)
+ {
+         seamless_window *sw;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw->id == id)
+                         return sw;
+         }
+         return NULL;
+ }
+ static seamless_window *
+ sw_get_window_by_wnd(Window wnd)
+ {
+         seamless_window *sw;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw->wnd == wnd)
+                         return sw;
+         }
+         return NULL;
+ }
  static void
- mwm_hide_decorations(void)
+ sw_remove_window(seamless_window * win)
+ {
+         seamless_window *sw, **prevnext = &g_seamless_windows;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw == win)
+                 {
+                         *prevnext = sw->next;
+                         sw->group->refcnt--;
+                         if (sw->group->refcnt == 0)
+                         {
+                                 XDestroyWindow(g_display, sw->group->wnd);
+                                 xfree(sw->group);
+                         }
+                         xfree(sw->position_timer);
+                         xfree(sw);
+                         return;
+                 }
+                 prevnext = &sw->next;
+         }
+         return;
+ }
+ /* Move all windows except wnd to new desktop */
+ static void
+ sw_all_to_desktop(Window wnd, unsigned int desktop)
+ {
+         seamless_window *sw;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw->wnd == wnd)
+                         continue;
+                 if (sw->desktop != desktop)
+                 {
+                         ewmh_move_to_desktop(sw->wnd, desktop);
+                         sw->desktop = desktop;
+                 }
+         }
+ }
+ /* Send our position */
+ static void
+ sw_update_position(seamless_window * sw)
+ {
+         XWindowAttributes wa;
+         int x, y;
+         Window child_return;
+         unsigned int serial;
+         XGetWindowAttributes(g_display, sw->wnd, &wa);
+         XTranslateCoordinates(g_display, sw->wnd, wa.root,
+                               -wa.border_width, -wa.border_width, &x, &y, &child_return);
+         serial = seamless_send_position(sw->id, x, y, wa.width, wa.height, 0);
+         sw->outstanding_position = True;
+         sw->outpos_serial = serial;
+         sw->outpos_xoffset = x;
+         sw->outpos_yoffset = y;
+         sw->outpos_width = wa.width;
+         sw->outpos_height = wa.height;
+ }
+ /* Check if it's time to send our position */
+ static void
+ sw_check_timers()
+ {
+         seamless_window *sw;
+         struct timeval now;
+         gettimeofday(&now, NULL);
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (timerisset(sw->position_timer) && timercmp(sw->position_timer, &now, <))
+                 {
+                         timerclear(sw->position_timer);
+                         sw_update_position(sw);
+                 }
+         }
+ }
+ static void
+ sw_restack_window(seamless_window * sw, unsigned long behind)
+ {
+         seamless_window *sw_above;
+         /* Remove window from stack */
+         for (sw_above = g_seamless_windows; sw_above; sw_above = sw_above->next)
+         {
+                 if (sw_above->behind == sw->id)
+                         break;
+         }
+         if (sw_above)
+                 sw_above->behind = sw->behind;
+         /* And then add it at the new position */
+         for (sw_above = g_seamless_windows; sw_above; sw_above = sw_above->next)
+         {
+                 if (sw_above->behind == behind)
+                         break;
+         }
+         if (sw_above)
+                 sw_above->behind = sw->id;
+         sw->behind = behind;
+ }
+ static void
+ sw_handle_restack(seamless_window * sw)
+ {
+         Status status;
+         Window root, parent, *children;
+         unsigned int nchildren, i;
+         seamless_window *sw_below;
+         status = XQueryTree(g_display, RootWindowOfScreen(g_screen),
+                             &root, &parent, &children, &nchildren);
+         if (!status || !nchildren)
+                 return;
+         sw_below = NULL;
+         i = 0;
+         while (children[i] != sw->wnd)
+         {
+                 i++;
+                 if (i >= nchildren)
+                         goto end;
+         }
+         for (i++; i < nchildren; i++)
+         {
+                 sw_below = sw_get_window_by_wnd(children[i]);
+                 if (sw_below)
+                         break;
+         }
+         if (!sw_below && !sw->behind)
+                 goto end;
+         if (sw_below && (sw_below->id == sw->behind))
+                 goto end;
+         if (sw_below)
+         {
+                 seamless_send_zchange(sw->id, sw_below->id, 0);
+                 sw_restack_window(sw, sw_below->id);
+         }
+         else
+         {
+                 seamless_send_zchange(sw->id, 0, 0);
+                 sw_restack_window(sw, 0);
+         }
+       end:
+         XFree(children);
+ }
+ static seamless_group *
+ sw_find_group(unsigned long id, RD_BOOL dont_create)
+ {
+         seamless_window *sw;
+         seamless_group *sg;
+         XSetWindowAttributes attribs;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw->group->id == id)
+                         return sw->group;
+         }
+         if (dont_create)
+                 return NULL;
+         sg = xmalloc(sizeof(seamless_group));
+         sg->wnd =
+                 XCreateWindow(g_display, RootWindowOfScreen(g_screen), -1, -1, 1, 1, 0,
+                               CopyFromParent, CopyFromParent, CopyFromParent, 0, &attribs);
+         sg->id = id;
+         sg->refcnt = 0;
+         return sg;
+ }
+ static void
+ mwm_hide_decorations(Window wnd)
  {
          PropMotifWmHints motif_hints;
          Atom hintsatom;
-Line 154 
 mwm_hide_decorations(void)
+Line 532 
 mwm_hide_decorations(void)
                  return;
          }
-         XChangeProperty(g_display, g_wnd, hintsatom, hintsatom, 32, PropModeReplace,
+         XChangeProperty(g_display, wnd, hintsatom, hintsatom, 32, PropModeReplace,
                          (unsigned char *) &motif_hints, PROP_MOTIF_WM_HINTS_ELEMENTS);
  }
- static PixelColour
+ typedef struct _sw_configurenotify_context
- split_colour15(uint32 colour)
+ {
+         Window window;
+         unsigned long serial;
+ } sw_configurenotify_context;
+ /* Predicate procedure for sw_wait_configurenotify */
+ static Bool
+ sw_configurenotify_p(Display * display, XEvent * xevent, XPointer arg)
+ {
+         sw_configurenotify_context *context = (sw_configurenotify_context *) arg;
+         if (xevent->xany.type == ConfigureNotify
+             && xevent->xconfigure.window == context->window
+             && xevent->xany.serial >= context->serial)
+                 return True;
+         return False;
+ }
+ /* Wait for a ConfigureNotify, with a equal or larger serial, on the
+    specified window. The event will be removed from the queue. We
+    could use XMaskEvent(StructureNotifyMask), but we would then risk
+    throwing away crucial events like DestroyNotify.
+    After a ConfigureWindow, according to ICCCM section 4.1.5, we
+    should recieve a ConfigureNotify, either a real or synthetic
+    one. This indicates that the configure has been "completed".
+    However, some WMs such as several versions of Metacity fails to
+    send synthetic events. See bug
+    http://bugzilla.gnome.org/show_bug.cgi?id=322840. We need to use a
+    timeout to avoid a hang. Tk uses the same approach. */
+ static void
+ sw_wait_configurenotify(Window wnd, unsigned long serial)
  {
-         PixelColour rv;
+         XEvent xevent;
-         rv.red = (colour & 0x7c00) >> 10;
+         sw_configurenotify_context context;
-         rv.red = (rv.red * 0xff) / 0x1f;
+         struct timeval now;
-         rv.green = (colour & 0x03e0) >> 5;
+         struct timeval nextsecond;
-         rv.green = (rv.green * 0xff) / 0x1f;
+         RD_BOOL got = False;
-         rv.blue = (colour & 0x1f);
-         rv.blue = (rv.blue * 0xff) / 0x1f;
+         context.window = wnd;
-         return rv;
+         context.serial = serial;
- }
+         gettimeofday(&nextsecond, NULL);
- static PixelColour
+         nextsecond.tv_sec += 1;
- split_colour16(uint32 colour)
- {
+         do
-         PixelColour rv;
+         {
-         rv.red = (colour & 0xf800) >> 11;
+                 if (XCheckIfEvent(g_display, &xevent, sw_configurenotify_p, (XPointer) & context))
-         rv.red = (rv.red * 0xff) / 0x1f;
+                 {
-         rv.green = (colour & 0x07e0) >> 5;
+                         got = True;
-         rv.green = (rv.green * 0xff) / 0x3f;
+                         break;
-         rv.blue = (colour & 0x001f);
+                 }
-         rv.blue = (rv.blue * 0xff) / 0x1f;
+                 usleep(100000);
-         return rv;
+                 gettimeofday(&now, NULL);
- }
+         }
+         while (timercmp(&now, &nextsecond, <));
- static PixelColour
- split_colour24(uint32 colour)
+         if (!got)
- {
+         {
-         PixelColour rv;
+                 warning("Broken Window Manager: Timeout while waiting for ConfigureNotify\n");
-         rv.blue = (colour & 0xff0000) >> 16;
+         }
-         rv.green = (colour & 0xff00) >> 8;
-         rv.red = (colour & 0xff);
-         return rv;
  }
- static uint32
+ /* Get the toplevel window, in case of reparenting */
- make_colour16(PixelColour pc)
+ static Window
+ sw_get_toplevel(Window wnd)
  {
-         pc.red = (pc.red * 0x1f) / 0xff;
+         Window root, parent;
-         pc.green = (pc.green * 0x3f) / 0xff;
+         Window *child_list;
-         pc.blue = (pc.blue * 0x1f) / 0xff;
+         unsigned int num_children;
-         return (pc.red << 11) | (pc.green << 5) | pc.blue;
+         while (1)
+         {
+                 XQueryTree(g_display, wnd, &root, &parent, &child_list, &num_children);
+                 if (root == parent)
+                 {
+                         break;
+                 }
+                 else if (!parent)
+                 {
+                         warning("Internal error: sw_get_toplevel called with root window\n");
+                 }
+                 wnd = parent;
+         }
+         return wnd;
  }
- static uint32
- make_colour24(PixelColour pc)
+ /* Check if wnd is already behind a window wrt stacking order */
+ static RD_BOOL
+ sw_window_is_behind(Window wnd, Window behind)
  {
-         if (g_xserver_be)
+         Window dummy1, dummy2;
+         Window *child_list;
+         unsigned int num_children;
+         unsigned int i;
+         RD_BOOL found_behind = False;
+         RD_BOOL found_wnd = False;
+         wnd = sw_get_toplevel(wnd);
+         behind = sw_get_toplevel(behind);
+         XQueryTree(g_display, RootWindowOfScreen(g_screen), &dummy1, &dummy2, &child_list,
+                    &num_children);
+         for (i = num_children - 1; i >= 0; i--)
          {
-                 return pc.red | (pc.green << 8) | (pc.blue << 16);
+                 if (child_list[i] == behind)
+                 {
+                         found_behind = True;
+                 }
+                 else if (child_list[i] == wnd)
+                 {
+                         found_wnd = True;
+                         break;
+                 }
          }
-         else
+         if (child_list)
+                 XFree(child_list);
+         if (!found_wnd)
          {
-                 return (pc.red << 16) | (pc.green << 8) | pc.blue;
+                 warning("sw_window_is_behind: Unable to find window 0x%lx\n", wnd);
+                 if (!found_behind)
+                 {
+                         warning("sw_window_is_behind: Unable to find behind window 0x%lx\n",
+                                 behind);
+                 }
          }
+         return found_behind;
  }
- static uint32
- make_colour32(PixelColour pc)
+ /* Test if the window manager correctly handles window restacking. In
+    particular, we are testing if it's possible to place a window
+    between two other windows. Many WMs such as Metacity can only stack
+    windows on the top or bottom. The window creation should mostly
+    match ui_seamless_create_window. */
+ static void
+ seamless_restack_test()
  {
-         if (g_xserver_be)
+         /* The goal is to have the middle window between top and
+            bottom.  The middle window is initially at the top,
+            though. */
+         Window wnds[3];         /* top, middle and bottom */
+         int i;
+         XEvent xevent;
+         XWindowChanges values;
+         unsigned long restack_serial;
+         for (i = 0; i < 3; i++)
+         {
+                 char name[64];
+                 wnds[i] =
+                         XCreateSimpleWindow(g_display, RootWindowOfScreen(g_screen), 0, 0, 20, 20,
+, 0, 0);
+                 snprintf(name, sizeof(name), "SeamlessRDP restack test - window %d", i);
+                 XStoreName(g_display, wnds[i], name);
+                 ewmh_set_wm_name(wnds[i], name);
+                 /* Hide decorations. Often this means that no
+                    reparenting will be done, which makes the restack
+                    easier. Besides, we want to mimic our other
+                    seamless windows as much as possible. We must still
+                    handle the case with reparenting, though. */
+                 mwm_hide_decorations(wnds[i]);
+                 /* Prevent windows from appearing in task bar */
+                 XSetTransientForHint(g_display, wnds[i], RootWindowOfScreen(g_screen));
+                 ewmh_set_window_popup(wnds[i]);
+                 /* We need to catch MapNotify/ConfigureNotify */
+                 XSelectInput(g_display, wnds[i], StructureNotifyMask);
+         }
+         /* Map Windows. Currently, we assume that XMapRaised places
+            the window on the top of the stack. Should be fairly safe;
+            the window is configured before it's mapped. */
+         XMapRaised(g_display, wnds[2]); /* bottom */
+         do
          {
-                 return pc.red | (pc.green << 8) | (pc.blue << 16);
+                 XWindowEvent(g_display, wnds[2], StructureNotifyMask, &xevent);
          }
-         else
+         while (xevent.type != MapNotify);
+         XMapRaised(g_display, wnds[0]); /* top */
+         do
+         {
+                 XWindowEvent(g_display, wnds[0], StructureNotifyMask, &xevent);
+         }
+         while (xevent.type != MapNotify);
+         XMapRaised(g_display, wnds[1]); /* middle */
+         do
          {
-                 return (pc.red << 16) | (pc.green << 8) | pc.blue;
+                 XWindowEvent(g_display, wnds[1], StructureNotifyMask, &xevent);
          }
+         while (xevent.type != MapNotify);
+         /* The stacking order should now be 1 - 0 - 2 */
+         if (!sw_window_is_behind(wnds[0], wnds[1]) || !sw_window_is_behind(wnds[2], wnds[1]))
+         {
+                 /* Ok, technically a WM is allowed to stack windows arbitrarily, but... */
+                 warning("Broken Window Manager: Unable to test window restacking\n");
+                 g_seamless_broken_restack = True;
+                 for (i = 0; i < 3; i++)
+                         XDestroyWindow(g_display, wnds[i]);
+                 return;
+         }
+         /* Restack, using XReconfigureWMWindow, which should correctly
+            handle reparented windows as well as nonreparenting WMs. */
+         values.stack_mode = Below;
+         values.sibling = wnds[0];
+         restack_serial = XNextRequest(g_display);
+         XReconfigureWMWindow(g_display, wnds[1], DefaultScreen(g_display), CWStackMode | CWSibling,
+                              &values);
+         sw_wait_configurenotify(wnds[1], restack_serial);
+         /* Now verify that middle is behind top but not behind
+            bottom */
+         if (!sw_window_is_behind(wnds[1], wnds[0]))
+         {
+                 warning("Broken Window Manager: doesn't handle restack (restack request was ignored)\n");
+                 g_seamless_broken_restack = True;
+         }
+         else if (sw_window_is_behind(wnds[1], wnds[2]))
+         {
+                 warning("Broken Window Manager: doesn't handle restack (window was moved to bottom)\n");
+                 g_seamless_broken_restack = True;
+         }
+         /* Destroy windows */
+         for (i = 0; i < 3; i++)
+                 XDestroyWindow(g_display, wnds[i]);
+ }
+ #define SPLITCOLOUR15(colour, rv) \
+ { \
+         rv.red = ((colour >> 7) & 0xf8) | ((colour >> 12) & 0x7); \
+         rv.green = ((colour >> 2) & 0xf8) | ((colour >> 8) & 0x7); \
+         rv.blue = ((colour << 3) & 0xf8) | ((colour >> 2) & 0x7); \
+ }
+ #define SPLITCOLOUR16(colour, rv) \
+ { \
+         rv.red = ((colour >> 8) & 0xf8) | ((colour >> 13) & 0x7); \
+         rv.green = ((colour >> 3) & 0xfc) | ((colour >> 9) & 0x3); \
+         rv.blue = ((colour << 3) & 0xf8) | ((colour >> 2) & 0x7); \
+ } \
+ #define SPLITCOLOUR24(colour, rv) \
+ { \
+         rv.blue = (colour & 0xff0000) >> 16; \
+         rv.green = (colour & 0x00ff00) >> 8; \
+         rv.red = (colour & 0x0000ff); \
  }
+ #define MAKECOLOUR(pc) \
+         ((pc.red >> g_red_shift_r) << g_red_shift_l) \
+                 | ((pc.green >> g_green_shift_r) << g_green_shift_l) \
+                 | ((pc.blue >> g_blue_shift_r) << g_blue_shift_l) \
  #define BSWAP16(x) { x = (((x & 0xff) << 8) | (x >> 8)); }
- #define BSWAP24(x) { x = (((x & 0xff) << 16) | (x >> 16) | ((x >> 8) & 0xff00)); }
+ #define BSWAP24(x) { x = (((x & 0xff) << 16) | (x >> 16) | (x & 0xff00)); }
  #define BSWAP32(x) { x = (((x & 0xff00ff) << 8) | ((x >> 8) & 0xff00ff)); \
                          x = (x << 16) | (x >> 16); }
+ /* The following macros output the same octet sequences
+    on both BE and LE hosts: */
+ #define BOUT16(o, x) { *(o++) = x >> 8; *(o++) = x; }
+ #define BOUT24(o, x) { *(o++) = x >> 16; *(o++) = x >> 8; *(o++) = x; }
+ #define BOUT32(o, x) { *(o++) = x >> 24; *(o++) = x >> 16; *(o++) = x >> 8; *(o++) = x; }
+ #define LOUT16(o, x) { *(o++) = x; *(o++) = x >> 8; }
+ #define LOUT24(o, x) { *(o++) = x; *(o++) = x >> 8; *(o++) = x >> 16; }
+ #define LOUT32(o, x) { *(o++) = x; *(o++) = x >> 8; *(o++) = x >> 16; *(o++) = x >> 24; }
  static uint32
  translate_colour(uint32 colour)
  {
-         switch (g_server_bpp)
+         PixelColour pc;
+         switch (g_server_depth)
          {
                  case 15:
-                         switch (g_bpp)
+                         SPLITCOLOUR15(colour, pc);
-                         {
-                                 case 16:
-                                         colour = make_colour16(split_colour15(colour));
-                                         break;
-                                 case 24:
-                                         colour = make_colour24(split_colour15(colour));
-                                         break;
-                                 case 32:
-                                         colour = make_colour32(split_colour15(colour));
-                                         break;
-                         }
                          break;
                  case 16:
-                         switch (g_bpp)
+                         SPLITCOLOUR16(colour, pc);
-                         {
-                                 case 16:
-                                         break;
-                                 case 24:
-                                         colour = make_colour24(split_colour16(colour));
-                                         break;
-                                 case 32:
-                                         colour = make_colour32(split_colour16(colour));
-                                         break;
-                         }
                          break;
                  case 24:
-                         switch (g_bpp)
+                 case 32:
-                         {
+                         SPLITCOLOUR24(colour, pc);
-                                 case 16:
+                         break;
-                                         colour = make_colour16(split_colour24(colour));
+                 default:
-                                         break;
+                         /* Avoid warning */
-                                 case 24:
+                         pc.red = 0;
-                                         break;
+                         pc.green = 0;
-                                 case 32:
+                         pc.blue = 0;
-                                         colour = make_colour32(split_colour24(colour));
-                                         break;
-                         }
                          break;
          }
-         return colour;
+         return MAKECOLOUR(pc);
+ }
+ /* indent is confused by UNROLL8 */
+ /* *INDENT-OFF* */
+ /* repeat and unroll, similar to bitmap.c */
+ /* potentialy any of the following translate */
+ /* functions can use repeat but just doing */
+ /* the most common ones */
+ #define UNROLL8(stm) { stm stm stm stm stm stm stm stm }
+ /* 2 byte output repeat */
+ #define REPEAT2(stm) \
+ { \
+         while (out <= end - 8 * 2) \
+                 UNROLL8(stm) \
+         while (out < end) \
+                 { stm } \
+ }
+ /* 3 byte output repeat */
+ #define REPEAT3(stm) \
+ { \
+         while (out <= end - 8 * 3) \
+                 UNROLL8(stm) \
+         while (out < end) \
+                 { stm } \
+ }
+ /* 4 byte output repeat */
+ #define REPEAT4(stm) \
+ { \
+         while (out <= end - 8 * 4) \
+                 UNROLL8(stm) \
+         while (out < end) \
+                 { stm } \
  }
+ /* *INDENT-ON* */
  static void
- translate8to8(uint8 * data, uint8 * out, uint8 * end)
+ translate8to8(const uint8 * data, uint8 * out, uint8 * end)
  {
          while (out < end)
                  *(out++) = (uint8) g_colmap[*(data++)];
  }
  static void
- translate8to16(uint8 * data, uint16 * out, uint16 * end)
+ translate8to16(const uint8 * data, uint8 * out, uint8 * end)
  {
-         while (out < end)
+         uint16 value;
-                 *(out++) = (uint16) g_colmap[*(data++)];
+         if (g_compatible_arch)
+         {
+                 /* *INDENT-OFF* */
+                 REPEAT2
+                 (
+                         *((uint16 *) out) = g_colmap[*(data++)];
+                         out += 2;
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
+         {
+                 while (out < end)
+                 {
+                         value = (uint16) g_colmap[*(data++)];
+                         BOUT16(out, value);
+                 }
+         }
+         else
+         {
+                 while (out < end)
+                 {
+                         value = (uint16) g_colmap[*(data++)];
+                         LOUT16(out, value);
+                 }
+         }
  }
  /* little endian - conversion happens when colourmap is built */
  static void
- translate8to24(uint8 * data, uint8 * out, uint8 * end)
+ translate8to24(const uint8 * data, uint8 * out, uint8 * end)
  {
          uint32 value;
-         while (out < end)
+         if (g_compatible_arch)
+         {
+                 while (out < end)
+                 {
+                         value = g_colmap[*(data++)];
+                         BOUT24(out, value);
+                 }
+         }
+         else
          {
-                 value = g_colmap[*(data++)];
+                 while (out < end)
-                 *(out++) = value;
+                 {
-                 *(out++) = value >> 8;
+                         value = g_colmap[*(data++)];
-                 *(out++) = value >> 16;
+                         LOUT24(out, value);
+                 }
          }
  }
  static void
- translate8to32(uint8 * data, uint32 * out, uint32 * end)
+ translate8to32(const uint8 * data, uint8 * out, uint8 * end)
  {
-         while (out < end)
+         uint32 value;
-                 *(out++) = g_colmap[*(data++)];
- }
- /* todo the remaining translate function might need some big endian check ?? */
+         if (g_compatible_arch)
+         {
+                 /* *INDENT-OFF* */
+                 REPEAT4
+                 (
+                         *((uint32 *) out) = g_colmap[*(data++)];
+                         out += 4;
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
+         {
+                 while (out < end)
+                 {
+                         value = g_colmap[*(data++)];
+                         BOUT32(out, value);
+                 }
+         }
+         else
+         {
+                 while (out < end)
+                 {
+                         value = g_colmap[*(data++)];
+                         LOUT32(out, value);
+                 }
+         }
+ }
  static void
- translate15to16(uint16 * data, uint8 * out, uint8 * end)
+ translate15to16(const uint16 * data, uint8 * out, uint8 * end)
  {
          uint16 pixel;
          uint16 value;
+         PixelColour pc;
-         while (out < end)
+         if (g_xserver_be)
          {
-                 pixel = *(data++);
+                 while (out < end)
-                 if (g_host_be)
-                 {
-                         BSWAP16(pixel)
-                 }
-                 value = make_colour16(split_colour15(pixel));
-                 if (g_xserver_be)
                  {
-                         *(out++) = value >> 8;
+                         pixel = *(data++);
-                         *(out++) = value;
+                         if (g_host_be)
+                         {
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         BOUT16(out, value);
                  }
-                 else
+         }
+         else
+         {
+                 while (out < end)
                  {
-                         *(out++) = value;
+                         pixel = *(data++);
-                         *(out++) = value >> 8;
+                         if (g_host_be)
+                         {
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         LOUT16(out, value);
                  }
          }
  }
  static void
- translate15to24(uint16 * data, uint8 * out, uint8 * end)
+ translate15to24(const uint16 * data, uint8 * out, uint8 * end)
  {
          uint32 value;
          uint16 pixel;
+         PixelColour pc;
-         while (out < end)
+         if (g_compatible_arch)
          {
-                 pixel = *(data++);
+                 /* *INDENT-OFF* */
+                 REPEAT3
-                 if (g_host_be)
+                 (
-                 {
+                         pixel = *(data++);
-                         BSWAP16(pixel)
+                         SPLITCOLOUR15(pixel, pc);
-                 }
+                         *(out++) = pc.blue;
+                         *(out++) = pc.green;
-                 value = make_colour24(split_colour15(pixel));
+                         *(out++) = pc.red;
-                 if (g_xserver_be)
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
+         {
+                 while (out < end)
                  {
-                         *(out++) = value >> 16;
+                         pixel = *(data++);
-                         *(out++) = value >> 8;
+                         if (g_host_be)
-                         *(out++) = value;
+                         {
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         BOUT24(out, value);
                  }
-                 else
+         }
+         else
+         {
+                 while (out < end)
                  {
-                         *(out++) = value;
+                         pixel = *(data++);
-                         *(out++) = value >> 8;
+                         if (g_host_be)
-                         *(out++) = value >> 16;
+                         {
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         LOUT24(out, value);
                  }
          }
  }
  static void
- translate15to32(uint16 * data, uint8 * out, uint8 * end)
+ translate15to32(const uint16 * data, uint8 * out, uint8 * end)
  {
          uint16 pixel;
          uint32 value;
+         PixelColour pc;
-         while (out < end)
+         if (g_compatible_arch)
          {
-                 pixel = *(data++);
+                 /* *INDENT-OFF* */
+                 REPEAT4
-                 if (g_host_be)
+                 (
-                 {
+                         pixel = *(data++);
-                         BSWAP16(pixel);
+                         SPLITCOLOUR15(pixel, pc);
-                 }
+                         *(out++) = pc.blue;
+                         *(out++) = pc.green;
-                 value = make_colour32(split_colour15(pixel));
+                         *(out++) = pc.red;
+                         *(out++) = 0;
-                 if (g_xserver_be)
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
+         {
+                 while (out < end)
                  {
-                         *(out++) = value >> 24;
+                         pixel = *(data++);
-                         *(out++) = value >> 16;
+                         if (g_host_be)
-                         *(out++) = value >> 8;
+                         {
-                         *(out++) = value;
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         BOUT32(out, value);
                  }
-                 else
+         }
+         else
+         {
+                 while (out < end)
                  {
-                         *(out++) = value;
+                         pixel = *(data++);
-                         *(out++) = value >> 8;
+                         if (g_host_be)
-                         *(out++) = value >> 16;
+                         {
-                         *(out++) = value >> 24;
+                                 BSWAP16(pixel);
+                         }
+                         SPLITCOLOUR15(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         LOUT32(out, value);
                  }
          }
  }
  static void
- translate16to16(uint16 * data, uint16 * out, uint16 * end)
+ translate16to16(const uint16 * data, uint8 * out, uint8 * end)
  {
+         uint16 pixel;
          uint16 value;
+         PixelColour pc;
          if (g_xserver_be)
          {
-                 while (out < end)
+                 if (g_host_be)
                  {
-                         value = *data;
+                         while (out < end)
-                         BSWAP16(value);
+                         {
-                         *out = value;
+                                 pixel = *(data++);
-                         data++;
+                                 BSWAP16(pixel);
-                         out++;
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 BOUT16(out, value);
+                         }
+                 }
+                 else
+                 {
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 BOUT16(out, value);
+                         }
                  }
          }
          else
          {
-                 while (out < end)
+                 if (g_host_be)
+                 {
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 BSWAP16(pixel);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT16(out, value);
+                         }
+                 }
+                 else
                  {
-                         *out = *data;
+                         while (out < end)
-                         out++;
+                         {
-                         data++;
+                                 pixel = *(data++);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT16(out, value);
+                         }
                  }
          }
  }
  static void
- translate16to24(uint16 * data, uint8 * out, uint8 * end)
+ translate16to24(const uint16 * data, uint8 * out, uint8 * end)
  {
          uint32 value;
          uint16 pixel;
+         PixelColour pc;
-         while (out < end)
+         if (g_compatible_arch)
+         {
+                 /* *INDENT-OFF* */
+                 REPEAT3
+                 (
+                         pixel = *(data++);
+                         SPLITCOLOUR16(pixel, pc);
+                         *(out++) = pc.blue;
+                         *(out++) = pc.green;
+                         *(out++) = pc.red;
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
          {
-                 pixel = *(data++);
                  if (g_host_be)
                  {
-                         BSWAP16(pixel)
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 BSWAP16(pixel);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 BOUT24(out, value);
+                         }
                  }
+                 else
-                 value = make_colour24(split_colour16(pixel));
+                 {
+                         while (out < end)
-                 if (g_xserver_be)
+                         {
+                                 pixel = *(data++);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 BOUT24(out, value);
+                         }
+                 }
+         }
+         else
+         {
+                 if (g_host_be)
                  {
-                         *(out++) = value >> 16;
+                         while (out < end)
-                         *(out++) = value >> 8;
+                         {
-                         *(out++) = value;
+                                 pixel = *(data++);
+                                 BSWAP16(pixel);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT24(out, value);
+                         }
                  }
                  else
                  {
-                         *(out++) = value;
+                         while (out < end)
-                         *(out++) = value >> 8;
+                         {
-                         *(out++) = value >> 16;
+                                 pixel = *(data++);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT24(out, value);
+                         }
                  }
          }
  }
  static void
- translate16to32(uint16 * data, uint8 * out, uint8 * end)
+ translate16to32(const uint16 * data, uint8 * out, uint8 * end)
  {
          uint16 pixel;
          uint32 value;
+         PixelColour pc;
-         while (out < end)
+         if (g_compatible_arch)
+         {
+                 /* *INDENT-OFF* */
+                 REPEAT4
+                 (
+                         pixel = *(data++);
+                         SPLITCOLOUR16(pixel, pc);
+                         *(out++) = pc.blue;
+                         *(out++) = pc.green;
+                         *(out++) = pc.red;
+                         *(out++) = 0;
+                 )
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
          {
-                 pixel = *(data++);
                  if (g_host_be)
                  {
-                         BSWAP16(pixel)
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 BSWAP16(pixel);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 BOUT32(out, value);
+                         }
                  }
+                 else
-                 value = make_colour32(split_colour16(pixel));
-                 if (g_xserver_be)
                  {
-                         *(out++) = value >> 24;
+                         while (out < end)
-                         *(out++) = value >> 16;
+                         {
-                         *(out++) = value >> 8;
+                                 pixel = *(data++);
-                         *(out++) = value;
+                                 SPLITCOLOUR16(pixel, pc);
-                 }
+                                 value = MAKECOLOUR(pc);
-                 else
+                                 BOUT32(out, value);
-                 {
+                         }
-                         *(out++) = value;
+                 }
-                         *(out++) = value >> 8;
+         }
-                         *(out++) = value >> 16;
+         else
-                         *(out++) = value >> 24;
+         {
-                 }
+                 if (g_host_be)
-         }
+                 {
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 BSWAP16(pixel);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT32(out, value);
+                         }
+                 }
+                 else
+                 {
+                         while (out < end)
+                         {
+                                 pixel = *(data++);
+                                 SPLITCOLOUR16(pixel, pc);
+                                 value = MAKECOLOUR(pc);
+                                 LOUT32(out, value);
+                         }
+                 }
+         }
  }
  static void
- translate24to16(uint8 * data, uint8 * out, uint8 * end)
+ translate24to16(const uint8 * data, uint8 * out, uint8 * end)
  {
          uint32 pixel = 0;
          uint16 value;
+         PixelColour pc;
          while (out < end)
          {
                  pixel = *(data++) << 16;
                  pixel |= *(data++) << 8;
                  pixel |= *(data++);
+                 SPLITCOLOUR24(pixel, pc);
-                 value = (uint16) make_colour16(split_colour24(pixel));
+                 value = MAKECOLOUR(pc);
                  if (g_xserver_be)
                  {
-                         *(out++) = value >> 8;
+                         BOUT16(out, value);
-                         *(out++) = value;
                  }
                  else
                  {
-                         *(out++) = value;
+                         LOUT16(out, value);
-                         *(out++) = value >> 8;
                  }
          }
  }
  static void
- translate24to24(uint8 * data, uint8 * out, uint8 * end)
+ translate24to24(const uint8 * data, uint8 * out, uint8 * end)
  {
-         while (out < end)
+         uint32 pixel;
+         uint32 value;
+         PixelColour pc;
+         if (g_xserver_be)
+         {
+                 while (out < end)
+                 {
+                         pixel = *(data++) << 16;
+                         pixel |= *(data++) << 8;
+                         pixel |= *(data++);
+                         SPLITCOLOUR24(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         BOUT24(out, value);
+                 }
+         }
+         else
          {
-                 *(out++) = (*(data++));
+                 while (out < end)
+                 {
+                         pixel = *(data++) << 16;
+                         pixel |= *(data++) << 8;
+                         pixel |= *(data++);
+                         SPLITCOLOUR24(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         LOUT24(out, value);
+                 }
          }
  }
  static void
- translate24to32(uint8 * data, uint8 * out, uint8 * end)
+ translate24to32(const uint8 * data, uint8 * out, uint8 * end)
  {
-         while (out < end)
+         uint32 pixel;
+         uint32 value;
+         PixelColour pc;
+         if (g_compatible_arch)
          {
-                 if (g_xserver_be)
+                 /* *INDENT-OFF* */
-                 {
+ #ifdef NEED_ALIGN
-                         *(out++) = 0x00;
+                 REPEAT4
+                 (
                          *(out++) = *(data++);
                          *(out++) = *(data++);
                          *(out++) = *(data++);
+                         *(out++) = 0;
+                 )
+ #else
+                 REPEAT4
+                 (
+                  /* Only read 3 bytes. Reading 4 bytes means reading beyond buffer. */
+                  *((uint32 *) out) = *((uint16 *) data) + (*((uint8 *) data + 2) << 16);
+                  out += 4;
+                  data += 3;
+                 )
+ #endif
+                 /* *INDENT-ON* */
+         }
+         else if (g_xserver_be)
+         {
+                 while (out < end)
+                 {
+                         pixel = *(data++) << 16;
+                         pixel |= *(data++) << 8;
+                         pixel |= *(data++);
+                         SPLITCOLOUR24(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         BOUT32(out, value);
                  }
-                 else
+         }
+         else
+         {
+                 while (out < end)
                  {
-                         *(out++) = *(data++);
+                         pixel = *(data++) << 16;
-                         *(out++) = *(data++);
+                         pixel |= *(data++) << 8;
-                         *(out++) = *(data++);
+                         pixel |= *(data++);
-                         *(out++) = 0x00;
+                         SPLITCOLOUR24(pixel, pc);
+                         value = MAKECOLOUR(pc);
+                         LOUT32(out, value);
                  }
          }
  }
-Line 571 
 translate24to32(uint8 * data, uint8 * ou
+Line 1429 
 translate24to32(uint8 * data, uint8 * ou
  static uint8 *
  translate_image(int width, int height, uint8 * data)
  {
-         int size = width * height * g_bpp / 8;
+         int size;
-         uint8 *out = (uint8 *) xmalloc(size);
+         uint8 *out;
-         uint8 *end = out + size;
+         uint8 *end;
+         /*
+            If RDP depth and X Visual depths match,
+            and arch(endian) matches, no need to translate:
+            just return data.
+            Note: select_visual should've already ensured g_no_translate
+            is only set for compatible depths, but the RDP depth might've
+            changed during connection negotiations.
+          */
+         /* todo */
+         if (g_server_depth == 32 && g_depth == 24)
+         {
+                 return data;
+         }
+         if (g_no_translate_image)
+         {
+                 if ((g_depth == 15 && g_server_depth == 15) ||
+                     (g_depth == 16 && g_server_depth == 16) ||
+                     (g_depth == 24 && g_server_depth == 24))
+                         return data;
+         }
+         size = width * height * (g_bpp / 8);
+         out = (uint8 *) xmalloc(size);
+         end = out + size;
-         switch (g_server_bpp)
+         switch (g_server_depth)
          {
                  case 24:
                          switch (g_bpp)
-Line 601 
 translate_image(int width, int height, u
+Line 1486 
 translate_image(int width, int height, u
                                          translate16to24((uint16 *) data, out, end);
                                          break;
                                  case 16:
-                                         translate16to16((uint16 *) data, (uint16 *) out,
+                                         translate16to16((uint16 *) data, out, end);
-                                                         (uint16 *) end);
                                          break;
                          }
                          break;
-Line 627 
 translate_image(int width, int height, u
+Line 1511 
 translate_image(int width, int height, u
                                          translate8to8(data, out, end);
                                          break;
                                  case 16:
-                                         translate8to16(data, (uint16 *) out, (uint16 *) end);
+                                         translate8to16(data, out, end);
                                          break;
                                  case 24:
                                          translate8to24(data, out, end);
                                          break;
                                  case 32:
-                                         translate8to32(data, (uint32 *) out, (uint32 *) end);
+                                         translate8to32(data, out, end);
                                          break;
                          }
                          break;
-Line 641 
 translate_image(int width, int height, u
+Line 1525 
 translate_image(int width, int height, u
          return out;
  }
- BOOL
+ static void
+ xwin_refresh_pointer_map(void)
+ {
+         unsigned char phys_to_log_map[sizeof(g_pointer_log_to_phys_map)];
+         int i, pointer_buttons;
+         pointer_buttons = XGetPointerMapping(g_display, phys_to_log_map, sizeof(phys_to_log_map));
+         if (pointer_buttons > sizeof(phys_to_log_map))
+                 pointer_buttons = sizeof(phys_to_log_map);
+         /* if multiple physical buttons map to the same logical button, then
+          * use the lower numbered physical one */
+         for (i = pointer_buttons - 1; i >= 0; i--)
+         {
+                 /* a user could specify arbitrary values for the logical button
+                  * number, ignore any that are abnormally large */
+                 if (phys_to_log_map[i] > sizeof(g_pointer_log_to_phys_map))
+                         continue;
+                 g_pointer_log_to_phys_map[phys_to_log_map[i] - 1] = i + 1;
+         }
+ }
+ RD_BOOL
  get_key_state(unsigned int state, uint32 keysym)
  {
          int modifierpos, key, keysymMask = 0;
-Line 666 
 get_key_state(unsigned int state, uint32
+Line 1572 
 get_key_state(unsigned int state, uint32
          return (state & keysymMask) ? True : False;
  }
- BOOL
+ static void
- ui_init(void)
+ calculate_shifts(uint32 mask, int *shift_r, int *shift_l)
+ {
+         *shift_l = ffs(mask) - 1;
+         mask >>= *shift_l;
+         *shift_r = 8 - ffs(mask & ~(mask >> 1));
+ }
+ /* Given a mask of a colour channel (e.g. XVisualInfo.red_mask),
+    calculates the bits-per-pixel of this channel (a.k.a. colour weight).
+  */
+ static unsigned
+ calculate_mask_weight(uint32 mask)
+ {
+         unsigned weight = 0;
+         do
+         {
+                 weight += (mask & 1);
+         }
+         while (mask >>= 1);
+         return weight;
+ }
+ static RD_BOOL
+ select_visual(int screen_num)
  {
          XPixmapFormatValues *pfm;
-         uint16 test;
+         int pixmap_formats_count, visuals_count;
+         XVisualInfo *vmatches = NULL;
+         XVisualInfo template;
          int i;
+         unsigned red_weight, blue_weight, green_weight;
-         g_display = XOpenDisplay(NULL);
+         red_weight = blue_weight = green_weight = 0;
-         if (g_display == NULL)
+         if (g_server_depth == -1)
          {
-                 error("Failed to open display: %s\n", XDisplayName(NULL));
+                 g_server_depth = DisplayPlanes(g_display, DefaultScreen(g_display));
+         }
+         pfm = XListPixmapFormats(g_display, &pixmap_formats_count);
+         if (pfm == NULL)
+         {
+                 error("Unable to get list of pixmap formats from display.\n");
+                 XCloseDisplay(g_display);
                  return False;
          }
-         g_x_socket = ConnectionNumber(g_display);
+         /* Search for best TrueColor visual */
-         g_screen = DefaultScreenOfDisplay(g_display);
+         template.class = TrueColor;
-         g_visual = DefaultVisualOfScreen(g_screen);
+         template.screen = screen_num;
-         g_depth = DefaultDepthOfScreen(g_screen);
+         vmatches =
+                 XGetVisualInfo(g_display, VisualClassMask | VisualScreenMask, &template,
+                                &visuals_count);
+         g_visual = NULL;
+         g_no_translate_image = False;
+         g_compatible_arch = False;
+         if (vmatches != NULL)
+         {
+                 for (i = 0; i < visuals_count; ++i)
+                 {
+                         XVisualInfo *visual_info = &vmatches[i];
+                         RD_BOOL can_translate_to_bpp = False;
+                         int j;
+                         /* Try to find a no-translation visual that'll
+                            allow us to use RDP bitmaps directly as ZPixmaps. */
+                         if (!g_xserver_be && (((visual_info->depth == 15) &&
+                                                /* R5G5B5 */
+                                                (visual_info->red_mask == 0x7c00) &&
+                                                (visual_info->green_mask == 0x3e0) &&
+                                                (visual_info->blue_mask == 0x1f)) ||
+                                               ((visual_info->depth == 16) &&
+                                                /* R5G6B5 */
+                                                (visual_info->red_mask == 0xf800) &&
+                                                (visual_info->green_mask == 0x7e0) &&
+                                                (visual_info->blue_mask == 0x1f)) ||
+                                               ((visual_info->depth == 24) &&
+                                                /* R8G8B8 */
+                                                (visual_info->red_mask == 0xff0000) &&
+                                                (visual_info->green_mask == 0xff00) &&
+                                                (visual_info->blue_mask == 0xff))))
+                         {
+                                 g_visual = visual_info->visual;
+                                 g_depth = visual_info->depth;
+                                 g_compatible_arch = !g_host_be;
+                                 g_no_translate_image = (visual_info->depth == g_server_depth);
+                                 if (g_no_translate_image)
+                                         /* We found the best visual */
+                                         break;
+                         }
+                         else
+                         {
+                                 g_compatible_arch = False;
+                         }
+                         if (visual_info->depth > 24)
+                         {
+                                 /* Avoid 32-bit visuals and likes like the plague.
+                                    They're either untested or proven to work bad
+                                    (e.g. nvidia's Composite 32-bit visual).
+                                    Most implementation offer a 24-bit visual anyway. */
+                                 continue;
+                         }
+                         /* Only care for visuals, for whose BPPs (not depths!)
+                            we have a translateXtoY function. */
+                         for (j = 0; j < pixmap_formats_count; ++j)
+                         {
+                                 if (pfm[j].depth == visual_info->depth)
+                                 {
+                                         if ((pfm[j].bits_per_pixel == 16) ||
+                                             (pfm[j].bits_per_pixel == 24) ||
+                                             (pfm[j].bits_per_pixel == 32))
+                                         {
+                                                 can_translate_to_bpp = True;
+                                         }
+                                         break;
+                                 }
+                         }
+                         /* Prefer formats which have the most colour depth.
+                            We're being truly aristocratic here, minding each
+                            weight on its own. */
+                         if (can_translate_to_bpp)
+                         {
+                                 unsigned vis_red_weight =
+                                         calculate_mask_weight(visual_info->red_mask);
+                                 unsigned vis_green_weight =
+                                         calculate_mask_weight(visual_info->green_mask);
+                                 unsigned vis_blue_weight =
+                                         calculate_mask_weight(visual_info->blue_mask);
+                                 if ((vis_red_weight >= red_weight)
+                                     && (vis_green_weight >= green_weight)
+                                     && (vis_blue_weight >= blue_weight))
+                                 {
+                                         red_weight = vis_red_weight;
+                                         green_weight = vis_green_weight;
+                                         blue_weight = vis_blue_weight;
+                                         g_visual = visual_info->visual;
+                                         g_depth = visual_info->depth;
+                                 }
+                         }
+                 }
+                 XFree(vmatches);
+         }
+         if (g_visual != NULL)
+         {
+                 g_owncolmap = False;
+                 calculate_shifts(g_visual->red_mask, &g_red_shift_r, &g_red_shift_l);
+                 calculate_shifts(g_visual->green_mask, &g_green_shift_r, &g_green_shift_l);
+                 calculate_shifts(g_visual->blue_mask, &g_blue_shift_r, &g_blue_shift_l);
+         }
+         else
+         {
+                 template.class = PseudoColor;
+                 template.depth = 8;
+                 template.colormap_size = 256;
+                 vmatches =
+                         XGetVisualInfo(g_display,
+                                        VisualClassMask | VisualDepthMask | VisualColormapSizeMask,
+                                        &template, &visuals_count);
+                 if (vmatches == NULL)
+                 {
+                         error("No usable TrueColor or PseudoColor visuals on this display.\n");
+                         XCloseDisplay(g_display);
+                         XFree(pfm);
+                         return False;
+                 }
+                 /* we use a colourmap, so the default visual should do */
+                 g_owncolmap = True;
+                 g_visual = vmatches[0].visual;
+                 g_depth = vmatches[0].depth;
+         }
-         pfm = XListPixmapFormats(g_display, &i);
+         g_bpp = 0;
-         if (pfm != NULL)
+         for (i = 0; i < pixmap_formats_count; ++i)
          {
-                 /* Use maximum bpp for this depth - this is generally
+                 XPixmapFormatValues *pf = &pfm[i];
-                    desirable, e.g. 24 bits->32 bits. */
+                 if (pf->depth == g_depth)
-                 while (i--)
                  {
-                         if ((pfm[i].depth == g_depth) && (pfm[i].bits_per_pixel > g_bpp))
+                         g_bpp = pf->bits_per_pixel;
+                         if (g_no_translate_image)
                          {
-                                 g_bpp = pfm[i].bits_per_pixel;
+                                 switch (g_server_depth)
+                                 {
+                                         case 15:
+                                         case 16:
+                                                 if (g_bpp != 16)
+                                                         g_no_translate_image = False;
+                                                 break;
+                                         case 24:
+                                                 /* Yes, this will force image translation
+                                                    on most modern servers which use 32 bits
+                                                    for R8G8B8. */
+                                                 if (g_bpp != 24)
+                                                         g_no_translate_image = False;
+                                                 break;
+                                         default:
+                                                 g_no_translate_image = False;
+                                                 break;
+                                 }
                          }
+                         /* Pixmap formats list is a depth-to-bpp mapping --
+                            there's just a single entry for every depth,
+                            so we can safely break here */
+                         break;
                  }
-                 XFree(pfm);
          }
+         XFree(pfm);
+         pfm = NULL;
+         return True;
+ }
-         if (g_bpp < 8)
+ static XErrorHandler g_old_error_handler;
+ static RD_BOOL g_error_expected = False;
+ /* Check if the X11 window corresponding to a seamless window with
+    specified id exists. */
+ RD_BOOL
+ sw_window_exists(unsigned long id)
+ {
+         seamless_window *sw;
+         char *name;
+         Status sts = 0;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+                 return False;
+         g_error_expected = True;
+         sts = XFetchName(g_display, sw->wnd, &name);
+         g_error_expected = False;
+         if (sts)
          {
-                 error("Less than 8 bpp not currently supported.\n");
+                 XFree(name);
-                 XCloseDisplay(g_display);
+         }
+         return sts;
+ }
+ static int
+ error_handler(Display * dpy, XErrorEvent * eev)
+ {
+         if (g_error_expected)
+                 return 0;
+         return g_old_error_handler(dpy, eev);
+ }
+ RD_BOOL
+ ui_init(void)
+ {
+         int screen_num;
+         g_display = XOpenDisplay(NULL);
+         if (g_display == NULL)
+         {
+                 error("Failed to open display: %s\n", XDisplayName(NULL));
                  return False;
          }
-         if (g_owncolmap != True)
          {
-                 g_xcolmap = DefaultColormapOfScreen(g_screen);
+                 uint16 endianess_test = 1;
-                 if (g_depth <= 8)
+                 g_host_be = !(RD_BOOL) (*(uint8 *) (&endianess_test));
-                         warning("Screen depth is 8 bits or lower: you may want to use -C for a private colourmap\n");
+         }
+         g_old_error_handler = XSetErrorHandler(error_handler);
+         g_xserver_be = (ImageByteOrder(g_display) == MSBFirst);
+         screen_num = DefaultScreen(g_display);
+         g_x_socket = ConnectionNumber(g_display);
+         g_screen = ScreenOfDisplay(g_display, screen_num);
+         g_depth = DefaultDepthOfScreen(g_screen);
+         if (!select_visual(screen_num))
+                 return False;
+         if (g_no_translate_image)
+         {
+                 DEBUG(("Performance optimization possible: avoiding image translation (colour depth conversion).\n"));
          }
-         g_gc = XCreateGC(g_display, RootWindowOfScreen(g_screen), 0, NULL);
+         if (g_server_depth > g_bpp)
+         {
+                 warning("Remote desktop colour depth %d higher than display colour depth %d.\n",
+                         g_server_depth, g_bpp);
+         }
-         if (DoesBackingStore(g_screen) != Always)
+         DEBUG(("RDP depth: %d, display depth: %d, display bpp: %d, X server BE: %d, host BE: %d\n",
-                 g_ownbackstore = True;
+                g_server_depth, g_depth, g_bpp, g_xserver_be, g_host_be));
-         test = 1;
+         if (!g_owncolmap)
-         g_host_be = !(BOOL) (*(uint8 *) (&test));
+         {
-         g_xserver_be = (ImageByteOrder(g_display) == MSBFirst);
+                 g_xcolmap =
+                         XCreateColormap(g_display, RootWindowOfScreen(g_screen), g_visual,
+                                         AllocNone);
+                 if (g_depth <= 8)
+                         warning("Display colour depth is %d bit: you may want to use -C for a private colourmap.\n", g_depth);
+         }
+         if ((!g_ownbackstore) && (DoesBackingStore(g_screen) != Always))
+         {
+                 warning("External BackingStore not available. Using internal.\n");
+                 g_ownbackstore = True;
+         }
-         if ((g_width == 0) || (g_height == 0))
+         /*
+          * Determine desktop size
+          */
+         if (g_fullscreen)
+         {
+                 g_width = WidthOfScreen(g_screen);
+                 g_height = HeightOfScreen(g_screen);
+                 g_using_full_workarea = True;
+         }
+         else if (g_width < 0)
+         {
+                 /* Percent of screen */
+                 if (-g_width >= 100)
+                         g_using_full_workarea = True;
+                 g_height = HeightOfScreen(g_screen) * (-g_width) / 100;
+                 g_width = WidthOfScreen(g_screen) * (-g_width) / 100;
+         }
+         else if (g_width == 0)
          {
                  /* Fetch geometry from _NET_WORKAREA */
                  uint32 x, y, cx, cy;
                  if (get_current_workarea(&x, &y, &cx, &cy) == 0)
                  {
                          g_width = cx;
                          g_height = cy;
+                         g_using_full_workarea = True;
                  }
                  else
                  {
                          warning("Failed to get workarea: probably your window manager does not support extended hints\n");
-                         g_width = 800;
+                         g_width = WidthOfScreen(g_screen);
-                         g_height = 600;
+                         g_height = HeightOfScreen(g_screen);
                  }
          }
-         if (g_fullscreen)
-         {
-                 g_width = WidthOfScreen(g_screen);
-                 g_height = HeightOfScreen(g_screen);
-         }
          /* make sure width is a multiple of 4 */
          g_width = (g_width + 3) & ~3;
-         if (g_ownbackstore)
-         {
-                 g_backstore =
-                         XCreatePixmap(g_display, RootWindowOfScreen(g_screen), g_width, g_height,
-                                       g_depth);
-                 /* clear to prevent rubbish being exposed at startup */
-                 XSetForeground(g_display, g_gc, BlackPixelOfScreen(g_screen));
-                 XFillRectangle(g_display, g_backstore, g_gc, 0, 0, g_width, g_height);
-         }
          g_mod_map = XGetModifierMapping(g_display);
+         xwin_refresh_pointer_map();
          xkeymap_init();
-Line 769 
 ui_init(void)
+Line 1929 
 ui_init(void)
                  g_IM = XOpenIM(g_display, NULL, NULL, NULL);
          xclip_init();
+         ewmh_init();
+         if (g_seamless_rdp)
+         {
+                 seamless_init();
+         }
-         /* todo take this out when high colour is done */
+         DEBUG_RDP5(("server bpp %d client bpp %d depth %d\n", g_server_depth, g_bpp, g_depth));
-         printf("server bpp %d client bpp %d depth %d\n", g_server_bpp, g_bpp, g_depth);
          return True;
  }
-Line 779 
 ui_init(void)
+Line 1943 
 ui_init(void)
  void
  ui_deinit(void)
  {
+         while (g_seamless_windows)
+         {
+                 XDestroyWindow(g_display, g_seamless_windows->wnd);
+                 sw_remove_window(g_seamless_windows);
+         }
+         xclip_deinit();
          if (g_IM != NULL)
                  XCloseIM(g_IM);
+         if (g_null_cursor != NULL)
+                 ui_destroy_cursor(g_null_cursor);
          XFreeModifiermap(g_mod_map);
          if (g_ownbackstore)
-Line 792 
 ui_deinit(void)
+Line 1967 
 ui_deinit(void)
          g_display = NULL;
  }
- BOOL
+ static void
+ get_window_attribs(XSetWindowAttributes * attribs)
+ {
+         attribs->background_pixel = BlackPixelOfScreen(g_screen);
+         attribs->background_pixel = WhitePixelOfScreen(g_screen);
+         attribs->border_pixel = WhitePixelOfScreen(g_screen);
+         attribs->backing_store = g_ownbackstore ? NotUseful : Always;
+         attribs->override_redirect = g_fullscreen;
+         attribs->colormap = g_xcolmap;
+ }
+ static void
+ get_input_mask(long *input_mask)
+ {
+         *input_mask = KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
+                 VisibilityChangeMask | FocusChangeMask | StructureNotifyMask;
+         if (g_sendmotion)
+                 *input_mask |= PointerMotionMask;
+         if (g_ownbackstore)
+                 *input_mask |= ExposureMask;
+         if (g_fullscreen || g_grab_keyboard)
+                 *input_mask |= EnterWindowMask;
+         if (g_grab_keyboard)
+                 *input_mask |= LeaveWindowMask;
+ }
+ RD_BOOL
  ui_create_window(void)
  {
+         uint8 null_pointer_mask[1] = { 0x80 };
+         uint8 null_pointer_data[24] = { 0x00 };
          XSetWindowAttributes attribs;
          XClassHint *classhints;
          XSizeHints *sizehints;
-Line 805 
 ui_create_window(void)
+Line 2011 
 ui_create_window(void)
          wndwidth = g_fullscreen ? WidthOfScreen(g_screen) : g_width;
          wndheight = g_fullscreen ? HeightOfScreen(g_screen) : g_height;
-         attribs.background_pixel = BlackPixelOfScreen(g_screen);
+         /* Handle -x-y portion of geometry string */
-         attribs.backing_store = g_ownbackstore ? NotUseful : Always;
+         if (g_xpos < 0 || (g_xpos == 0 && (g_pos & 2)))
-         attribs.override_redirect = g_fullscreen;
+                 g_xpos = WidthOfScreen(g_screen) + g_xpos - g_width;
+         if (g_ypos < 0 || (g_ypos == 0 && (g_pos & 4)))
-         g_wnd = XCreateWindow(g_display, RootWindowOfScreen(g_screen), 0, 0, wndwidth, wndheight,
+                 g_ypos = HeightOfScreen(g_screen) + g_ypos - g_height;
-, CopyFromParent, InputOutput, CopyFromParent,
-                               CWBackPixel | CWBackingStore | CWOverrideRedirect, &attribs);
+         get_window_attribs(&attribs);
+         g_wnd = XCreateWindow(g_display, RootWindowOfScreen(g_screen), g_xpos, g_ypos, wndwidth,
+                               wndheight, 0, g_depth, InputOutput, g_visual,
+                               CWBackPixel | CWBackingStore | CWOverrideRedirect | CWColormap |
+                               CWBorderPixel, &attribs);
+         if (g_gc == NULL)
+         {
+                 g_gc = XCreateGC(g_display, g_wnd, 0, NULL);
+                 ui_reset_clip();
+         }
+         if (g_create_bitmap_gc == NULL)
+                 g_create_bitmap_gc = XCreateGC(g_display, g_wnd, 0, NULL);
+         if ((g_ownbackstore) && (g_backstore == 0))
+         {
+                 g_backstore = XCreatePixmap(g_display, g_wnd, g_width, g_height, g_depth);
+                 /* clear to prevent rubbish being exposed at startup */
+                 XSetForeground(g_display, g_gc, BlackPixelOfScreen(g_screen));
+                 XFillRectangle(g_display, g_backstore, g_gc, 0, 0, g_width, g_height);
+         }
          XStoreName(g_display, g_wnd, g_title);
+         ewmh_set_wm_name(g_wnd, g_title);
          if (g_hide_decorations)
-                 mwm_hide_decorations();
+                 mwm_hide_decorations(g_wnd);
          classhints = XAllocClassHint();
          if (classhints != NULL)
-Line 830 
 ui_create_window(void)
+Line 2060 
 ui_create_window(void)
          if (sizehints)
          {
                  sizehints->flags = PMinSize | PMaxSize;
+                 if (g_pos)
+                         sizehints->flags |= PPosition;
                  sizehints->min_width = sizehints->max_width = g_width;
                  sizehints->min_height = sizehints->max_height = g_height;
                  XSetWMNormalHints(g_display, g_wnd, sizehints);
                  XFree(sizehints);
          }
-         input_mask = KeyPressMask | KeyReleaseMask | ButtonPressMask | ButtonReleaseMask |
+         if (g_embed_wnd)
-                 VisibilityChangeMask | FocusChangeMask;
+         {
+                 XReparentWindow(g_display, g_wnd, (Window) g_embed_wnd, 0, 0);
+         }
-         if (g_sendmotion)
+         get_input_mask(&input_mask);
-                 input_mask |= PointerMotionMask;
-         if (g_ownbackstore)
-                 input_mask |= ExposureMask;
-         if (g_fullscreen || g_grab_keyboard)
-                 input_mask |= EnterWindowMask;
-         if (g_grab_keyboard)
-                 input_mask |= LeaveWindowMask;
          if (g_IM != NULL)
          {
-Line 867 
 ui_create_window(void)
+Line 2094 
 ui_create_window(void)
                  XMaskEvent(g_display, VisibilityChangeMask, &xevent);
          }
          while (xevent.type != VisibilityNotify);
+         g_Unobscured = xevent.xvisibility.state == VisibilityUnobscured;
          g_focused = False;
          g_mouse_in_wnd = False;
-Line 876 
 ui_create_window(void)
+Line 2104 
 ui_create_window(void)
          g_kill_atom = XInternAtom(g_display, "WM_DELETE_WINDOW", True);
          XSetWMProtocols(g_display, g_wnd, &g_kill_atom, 1);
+         /* create invisible 1x1 cursor to be used as null cursor */
+         if (g_null_cursor == NULL)
+                 g_null_cursor = ui_create_cursor(0, 0, 1, 1, null_pointer_mask, null_pointer_data);
+         if (g_seamless_rdp)
+         {
+                 seamless_restack_test();
+         }
          return True;
  }
  void
+ ui_resize_window()
+ {
+         XSizeHints *sizehints;
+         Pixmap bs;
+         sizehints = XAllocSizeHints();
+         if (sizehints)
+         {
+                 sizehints->flags = PMinSize | PMaxSize;
+                 sizehints->min_width = sizehints->max_width = g_width;
+                 sizehints->min_height = sizehints->max_height = g_height;
+                 XSetWMNormalHints(g_display, g_wnd, sizehints);
+                 XFree(sizehints);
+         }
+         if (!(g_fullscreen || g_embed_wnd))
+         {
+                 XResizeWindow(g_display, g_wnd, g_width, g_height);
+         }
+         /* create new backstore pixmap */
+         if (g_backstore != 0)
+         {
+                 bs = XCreatePixmap(g_display, g_wnd, g_width, g_height, g_depth);
+                 XSetForeground(g_display, g_gc, BlackPixelOfScreen(g_screen));
+                 XFillRectangle(g_display, bs, g_gc, 0, 0, g_width, g_height);
+                 XCopyArea(g_display, g_backstore, bs, g_gc, 0, 0, g_width, g_height, 0, 0);
+                 XFreePixmap(g_display, g_backstore);
+                 g_backstore = bs;
+         }
+ }
+ void
  ui_destroy_window(void)
  {
          if (g_IC != NULL)
-Line 893 
 xwin_toggle_fullscreen(void)
+Line 2163 
 xwin_toggle_fullscreen(void)
  {
          Pixmap contents = 0;
+         if (g_seamless_active)
+                 /* Turn off SeamlessRDP mode */
+                 ui_seamless_toggle();
          if (!g_ownbackstore)
          {
                  /* need to save contents of window */
-Line 913 
 xwin_toggle_fullscreen(void)
+Line 2187 
 xwin_toggle_fullscreen(void)
          }
  }
- /* Process all events in Xlib queue
+ static void
+ handle_button_event(XEvent xevent, RD_BOOL down)
+ {
+         uint16 button, flags = 0;
+         g_last_gesturetime = xevent.xbutton.time;
+         /* Reverse the pointer button mapping, e.g. in the case of
+            "left-handed mouse mode"; the RDP session expects to
+            receive physical buttons (true in mstsc as well) and
+            logical button behavior depends on the remote desktop's own
+            mouse settings */
+         xevent.xbutton.button = g_pointer_log_to_phys_map[xevent.xbutton.button - 1];
+         button = xkeymap_translate_button(xevent.xbutton.button);
+         if (button == 0)
+                 return;
+         if (down)
+                 flags = MOUSE_FLAG_DOWN;
+         /* Stop moving window when button is released, regardless of cursor position */
+         if (g_moving_wnd && (xevent.type == ButtonRelease))
+                 g_moving_wnd = False;
+         /* If win_button_size is nonzero, enable single app mode */
+         if (xevent.xbutton.y < g_win_button_size)
+         {
+                 /*  Check from right to left: */
+                 if (xevent.xbutton.x >= g_width - g_win_button_size)
+                 {
+                         /* The close button, continue */
+                         ;
+                 }
+                 else if (xevent.xbutton.x >= g_width - g_win_button_size * 2)
+                 {
+                         /* The maximize/restore button. Do not send to
+                            server.  It might be a good idea to change the
+                            cursor or give some other visible indication
+                            that rdesktop inhibited this click */
+                         if (xevent.type == ButtonPress)
+                                 return;
+                 }
+                 else if (xevent.xbutton.x >= g_width - g_win_button_size * 3)
+                 {
+                         /* The minimize button. Iconify window. */
+                         if (xevent.type == ButtonRelease)
+                         {
+                                 /* Release the mouse button outside the minimize button, to prevent the
+                                    actual minimazation to happen */
+                                 rdp_send_input(time(NULL), RDP_INPUT_MOUSE, button, 1, 1);
+                                 XIconifyWindow(g_display, g_wnd, DefaultScreen(g_display));
+                                 return;
+                         }
+                 }
+                 else if (xevent.xbutton.x <= g_win_button_size)
+                 {
+                         /* The system menu. Ignore. */
+                         if (xevent.type == ButtonPress)
+                                 return;
+                 }
+                 else
+                 {
+                         /* The title bar. */
+                         if (xevent.type == ButtonPress)
+                         {
+                                 if (!g_fullscreen && g_hide_decorations && !g_using_full_workarea)
+                                 {
+                                         g_moving_wnd = True;
+                                         g_move_x_offset = xevent.xbutton.x;
+                                         g_move_y_offset = xevent.xbutton.y;
+                                 }
+                                 return;
+                         }
+                 }
+         }
+         if (xevent.xmotion.window == g_wnd)
+         {
+                 rdp_send_input(time(NULL), RDP_INPUT_MOUSE,
+                                flags | button, xevent.xbutton.x, xevent.xbutton.y);
+         }
+         else
+         {
+                 /* SeamlessRDP */
+                 rdp_send_input(time(NULL), RDP_INPUT_MOUSE,
+                                flags | button, xevent.xbutton.x_root, xevent.xbutton.y_root);
+         }
+ }
+ /* Process events in Xlib queue
     Returns 0 after user quit, 1 otherwise */
  static int
  xwin_process_events(void)
  {
          XEvent xevent;
          KeySym keysym;
-         uint16 button, flags;
          uint32 ev_time;
-         key_translation tr;
          char str[256];
          Status status;
-         unsigned int state;
+         int events = 0;
-         Window wdummy;
+         seamless_window *sw;
-         int dummy;
-         while (XPending(g_display) > 0)
+         while ((XPending(g_display) > 0) && events++ < 20)
          {
                  XNextEvent(g_display, &xevent);
-Line 939 
 xwin_process_events(void)
+Line 2298 
 xwin_process_events(void)
                          continue;
                  }
-                 flags = 0;
                  switch (xevent.type)
                  {
+                         case VisibilityNotify:
+                                 if (xevent.xvisibility.window == g_wnd)
+                                         g_Unobscured =
+                                                 xevent.xvisibility.state == VisibilityUnobscured;
+                                 break;
                          case ClientMessage:
                                  /* the window manager told us to quit */
                                  if ((xevent.xclient.message_type == g_protocol_atom)
                                      && ((Atom) xevent.xclient.data.l[0] == g_kill_atom))
-                                         /* Quit */
+                                 {
-                                         return 0;
+                                         /* When killing a seamless window, close the window on the
+                                            serverside instead of terminating rdesktop */
+                                         sw = sw_get_window_by_wnd(xevent.xclient.window);
+                                         if (!sw)
+                                                 /* Otherwise, quit */
+                                                 return 0;
+                                         /* send seamless destroy process message */
+                                         seamless_send_destroy(sw->id);
+                                 }
                                  break;
                          case KeyPress:
-Line 974 
 xwin_process_events(void)
+Line 2345 
 xwin_process_events(void)
                                                        str, sizeof(str), &keysym, NULL);
                                  }
-                                 DEBUG_KBD(("KeyPress for (keysym 0x%lx, %s)\n", keysym,
+                                 DEBUG_KBD(("KeyPress for keysym (0x%lx, %s)\n", keysym,
                                             get_ksname(keysym)));
                                  ev_time = time(NULL);
                                  if (handle_special_keys(keysym, xevent.xkey.state, ev_time, True))
                                          break;
-                                 tr = xkeymap_translate_key(keysym,
+                                 xkeymap_send_keys(keysym, xevent.xkey.keycode, xevent.xkey.state,
-                                                            xevent.xkey.keycode, xevent.xkey.state);
+                                                   ev_time, True, 0);
-                                 if (tr.scancode == 0)
-                                         break;
-                                 save_remote_modifiers(tr.scancode);
-                                 ensure_remote_modifiers(ev_time, tr);
-                                 rdp_send_scancode(ev_time, RDP_KEYPRESS, tr.scancode);
-                                 restore_remote_modifiers(ev_time, tr.scancode);
                                  break;
                          case KeyRelease:
-Line 999 
 xwin_process_events(void)
+Line 2361 
 xwin_process_events(void)
                                  XLookupString((XKeyEvent *) & xevent, str,
                                                sizeof(str), &keysym, NULL);
-                                 DEBUG_KBD(("\nKeyRelease for (keysym 0x%lx, %s)\n", keysym,
+                                 DEBUG_KBD(("\nKeyRelease for keysym (0x%lx, %s)\n", keysym,
                                             get_ksname(keysym)));
                                  ev_time = time(NULL);
                                  if (handle_special_keys(keysym, xevent.xkey.state, ev_time, False))
                                          break;
-                                 tr = xkeymap_translate_key(keysym,
+                                 xkeymap_send_keys(keysym, xevent.xkey.keycode, xevent.xkey.state,
-                                                            xevent.xkey.keycode, xevent.xkey.state);
+                                                   ev_time, False, 0);
-                                 if (tr.scancode == 0)
-                                         break;
-                                 rdp_send_scancode(ev_time, RDP_KEYRELEASE, tr.scancode);
                                  break;
                          case ButtonPress:
-                                 flags = MOUSE_FLAG_DOWN;
+                                 handle_button_event(xevent, True);
-                                 /* fall through */
+                                 break;
                          case ButtonRelease:
-                                 g_last_gesturetime = xevent.xbutton.time;
+                                 handle_button_event(xevent, False);
-                                 button = xkeymap_translate_button(xevent.xbutton.button);
-                                 if (button == 0)
-                                         break;
-                                 /* If win_button_size is nonzero, enable single app mode */
-                                 if (xevent.xbutton.y < g_win_button_size)
-                                 {
-                                         /* Stop moving window when button is released, regardless of cursor position */
-                                         if (g_moving_wnd && (xevent.type == ButtonRelease))
-                                                 g_moving_wnd = False;
-                                         /*  Check from right to left: */
-                                         if (xevent.xbutton.x >= g_width - g_win_button_size)
-                                         {
-                                                 /* The close button, continue */
-                                                 ;
-                                         }
-                                         else if (xevent.xbutton.x >=
-                                                  g_width - g_win_button_size * 2)
-                                         {
-                                                 /* The maximize/restore button. Do not send to
-                                                    server.  It might be a good idea to change the
-                                                    cursor or give some other visible indication
-                                                    that rdesktop inhibited this click */
-                                                 break;
-                                         }
-                                         else if (xevent.xbutton.x >=
-                                                  g_width - g_win_button_size * 3)
-                                         {
-                                                 /* The minimize button. Iconify window. */
-                                                 XIconifyWindow(g_display, g_wnd,
-                                                                DefaultScreen(g_display));
-                                                 break;
-                                         }
-                                         else if (xevent.xbutton.x <= g_win_button_size)
-                                         {
-                                                 /* The system menu. Ignore. */
-                                                 break;
-                                         }
-                                         else
-                                         {
-                                                 /* The title bar. */
-                                                 if ((xevent.type == ButtonPress) && !g_fullscreen
-                                                     && g_hide_decorations)
-                                                 {
-                                                         g_moving_wnd = True;
-                                                         g_move_x_offset = xevent.xbutton.x;
-                                                         g_move_y_offset = xevent.xbutton.y;
-                                                 }
-                                                 break;
-                                         }
-                                 }
-                                 rdp_send_input(time(NULL), RDP_INPUT_MOUSE,
-                                                flags | button, xevent.xbutton.x, xevent.xbutton.y);
                                  break;
                          case MotionNotify:
-Line 1092 
 xwin_process_events(void)
+Line 2392 
 xwin_process_events(void)
                                  if (g_fullscreen && !g_focused)
                                          XSetInputFocus(g_display, g_wnd, RevertToPointerRoot,
                                                         CurrentTime);
-                                 rdp_send_input(time(NULL), RDP_INPUT_MOUSE,
-                                                MOUSE_FLAG_MOVE, xevent.xmotion.x, xevent.xmotion.y);
+                                 if (xevent.xmotion.window == g_wnd)
+                                 {
+                                         rdp_send_input(time(NULL), RDP_INPUT_MOUSE, MOUSE_FLAG_MOVE,
+                                                        xevent.xmotion.x, xevent.xmotion.y);
+                                 }
+                                 else
+                                 {
+                                         /* SeamlessRDP */
+                                         rdp_send_input(time(NULL), RDP_INPUT_MOUSE, MOUSE_FLAG_MOVE,
+                                                        xevent.xmotion.x_root,
+                                                        xevent.xmotion.y_root);
+                                 }
                                  break;
                          case FocusIn:
                                  if (xevent.xfocus.mode == NotifyGrab)
                                          break;
                                  g_focused = True;
-                                 XQueryPointer(g_display, g_wnd, &wdummy, &wdummy, &dummy, &dummy,
+                                 reset_modifier_keys();
-                                               &dummy, &dummy, &state);
-                                 reset_modifier_keys(state);
                                  if (g_grab_keyboard && g_mouse_in_wnd)
                                          XGrabKeyboard(g_display, g_wnd, True,
                                                        GrabModeAsync, GrabModeAsync, CurrentTime);
+                                 sw = sw_get_window_by_wnd(xevent.xfocus.window);
+                                 if (!sw)
+                                         break;
+                                 /* Menu windows are real X11 windows,
+                                    with focus. When such a window is
+                                    destroyed, focus is reverted to the
+                                    main application window, which
+                                    would cause us to send FOCUS. This
+                                    breaks window switching in, say,
+                                    Seamonkey. We shouldn't need to
+                                    send FOCUS: Windows should also
+                                    revert focus to some other window
+                                    when the menu window is
+                                    destroyed. So, we only send FOCUS
+                                    if the previous focus window still
+                                    exists. */
+                                 if (sw->id != g_seamless_focused)
+                                 {
+                                         if (sw_window_exists(g_seamless_focused))
+                                                 seamless_send_focus(sw->id, 0);
+                                         g_seamless_focused = sw->id;
+                                 }
                                  break;
                          case FocusOut:
-Line 1138 
 xwin_process_events(void)
+Line 2472 
 xwin_process_events(void)
                                  break;
                          case Expose:
-                                 XCopyArea(g_display, g_backstore, g_wnd, g_gc,
+                                 if (xevent.xexpose.window == g_wnd)
-                                           xevent.xexpose.x, xevent.xexpose.y,
+                                 {
-                                           xevent.xexpose.width,
+                                         XCopyArea(g_display, g_backstore, xevent.xexpose.window,
-                                           xevent.xexpose.height,
+                                                   g_gc,
-                                           xevent.xexpose.x, xevent.xexpose.y);
+                                                   xevent.xexpose.x, xevent.xexpose.y,
+                                                   xevent.xexpose.width, xevent.xexpose.height,
+                                                   xevent.xexpose.x, xevent.xexpose.y);
+                                 }
+                                 else
+                                 {
+                                         sw = sw_get_window_by_wnd(xevent.xexpose.window);
+                                         if (!sw)
+                                                 break;
+                                         XCopyArea(g_display, g_backstore,
+                                                   xevent.xexpose.window, g_gc,
+                                                   xevent.xexpose.x + sw->xoffset,
+                                                   xevent.xexpose.y + sw->yoffset,
+                                                   xevent.xexpose.width,
+                                                   xevent.xexpose.height, xevent.xexpose.x,
+                                                   xevent.xexpose.y);
+                                 }
                                  break;
                          case MappingNotify:
-Line 1157 
 xwin_process_events(void)
+Line 2508 
 xwin_process_events(void)
                                          XFreeModifiermap(g_mod_map);
                                          g_mod_map = XGetModifierMapping(g_display);
                                  }
+                                 if (xevent.xmapping.request == MappingPointer)
+                                 {
+                                         xwin_refresh_pointer_map();
+                                 }
                                  break;
                                  /* clipboard stuff */
-Line 1171 
 xwin_process_events(void)
+Line 2528 
 xwin_process_events(void)
                                  break;
                          case PropertyNotify:
                                  xclip_handle_PropertyNotify(&xevent.xproperty);
+                                 if (xevent.xproperty.window == g_wnd)
+                                         break;
+                                 if (xevent.xproperty.window == DefaultRootWindow(g_display))
+                                         break;
+                                 /* seamless */
+                                 sw = sw_get_window_by_wnd(xevent.xproperty.window);
+                                 if (!sw)
+                                         break;
+                                 if ((xevent.xproperty.atom == g_net_wm_state_atom)
+                                     && (xevent.xproperty.state == PropertyNewValue))
+                                 {
+                                         sw->state = ewmh_get_window_state(sw->wnd);
+                                         seamless_send_state(sw->id, sw->state, 0);
+                                 }
+                                 if ((xevent.xproperty.atom == g_net_wm_desktop_atom)
+                                     && (xevent.xproperty.state == PropertyNewValue))
+                                 {
+                                         sw->desktop = ewmh_get_window_desktop(sw->wnd);
+                                         sw_all_to_desktop(sw->wnd, sw->desktop);
+                                 }
+                                 break;
+                         case MapNotify:
+                                 if (!g_seamless_active)
+                                         rdp_send_client_window_status(1);
+                                 break;
+                         case UnmapNotify:
+                                 if (!g_seamless_active)
+                                         rdp_send_client_window_status(0);
+                                 break;
+                         case ConfigureNotify:
+                                 if (!g_seamless_active)
+                                         break;
+                                 sw = sw_get_window_by_wnd(xevent.xconfigure.window);
+                                 if (!sw)
+                                         break;
+                                 gettimeofday(sw->position_timer, NULL);
+                                 if (sw->position_timer->tv_usec + SEAMLESSRDP_POSITION_TIMER >=
+                                     1000000)
+                                 {
+                                         sw->position_timer->tv_usec +=
+                                                 SEAMLESSRDP_POSITION_TIMER - 1000000;
+                                         sw->position_timer->tv_sec += 1;
+                                 }
+                                 else
+                                 {
+                                         sw->position_timer->tv_usec += SEAMLESSRDP_POSITION_TIMER;
+                                 }
+                                 sw_handle_restack(sw);
                                  break;
                  }
          }
-Line 1182 
 xwin_process_events(void)
+Line 2594 
 xwin_process_events(void)
  int
  ui_select(int rdp_socket)
  {
-         int n = (rdp_socket > g_x_socket) ? rdp_socket + 1 : g_x_socket + 1;
+         int n;
          fd_set rfds, wfds;
+         struct timeval tv;
+         RD_BOOL s_timeout = False;
          while (True)
          {
+                 n = (rdp_socket > g_x_socket) ? rdp_socket : g_x_socket;
                  /* Process any events already waiting */
                  if (!xwin_process_events())
                          /* User quit */
                          return 0;
+                 if (g_seamless_active)
+                         sw_check_timers();
                  FD_ZERO(&rfds);
                  FD_ZERO(&wfds);
                  FD_SET(rdp_socket, &rfds);
                  FD_SET(g_x_socket, &rfds);
+                 /* default timeout */
+                 tv.tv_sec = 60;
+                 tv.tv_usec = 0;
  #ifdef WITH_RDPSND
-                 /* FIXME: there should be an API for registering fds */
+                 rdpsnd_add_fds(&n, &rfds, &wfds, &tv);
-                 if (g_dsp_busy)
-                 {
-                         FD_SET(g_dsp_fd, &wfds);
-                         n = (g_dsp_fd + 1 > n) ? g_dsp_fd + 1 : n;
-                 }
  #endif
-                 switch (select(n, &rfds, &wfds, NULL, NULL))
+                 /* add redirection handles */
+                 rdpdr_add_fds(&n, &rfds, &wfds, &tv, &s_timeout);
+                 seamless_select_timeout(&tv);
+                 n++;
+                 switch (select(n, &rfds, &wfds, NULL, &tv))
                  {
                          case -1:
                                  error("select: %s\n", strerror(errno));
                          case 0:
+ #ifdef WITH_RDPSND
+                                 rdpsnd_check_fds(&rfds, &wfds);
+ #endif
+                                 /* Abort serial read calls */
+                                 if (s_timeout)
+                                         rdpdr_check_fds(&rfds, &wfds, (RD_BOOL) True);
                                  continue;
                  }
+ #ifdef WITH_RDPSND
+                 rdpsnd_check_fds(&rfds, &wfds);
+ #endif
+                 rdpdr_check_fds(&rfds, &wfds, (RD_BOOL) False);
                  if (FD_ISSET(rdp_socket, &rfds))
                          return 1;
- #ifdef WITH_RDPSND
-                 if (g_dsp_busy && FD_ISSET(g_dsp_fd, &wfds))
-                         wave_out_play();
- #endif
          }
  }
-Line 1231 
 ui_move_pointer(int x, int y)
+Line 2663 
 ui_move_pointer(int x, int y)
          XWarpPointer(g_display, g_wnd, g_wnd, 0, 0, 0, 0, x, y);
  }
- HBITMAP
+ RD_HBITMAP
  ui_create_bitmap(int width, int height, uint8 * data)
  {
          XImage *image;
          Pixmap bitmap;
          uint8 *tdata;
+         int bitmap_pad;
+         if (g_server_depth == 8)
+         {
+                 bitmap_pad = 8;
+         }
+         else
+         {
+                 bitmap_pad = g_bpp;
+                 if (g_bpp == 24)
+                         bitmap_pad = 32;
+         }
          tdata = (g_owncolmap ? data : translate_image(width, height, data));
          bitmap = XCreatePixmap(g_display, g_wnd, width, height, g_depth);
          image = XCreateImage(g_display, g_visual, g_depth, ZPixmap, 0,
-                              (char *) tdata, width, height, g_server_bpp == 8 ? 8 : g_bpp, 0);
+                              (char *) tdata, width, height, bitmap_pad, 0);
-         XPutImage(g_display, bitmap, g_gc, image, 0, 0, 0, 0, width, height);
+         XPutImage(g_display, bitmap, g_create_bitmap_gc, image, 0, 0, 0, 0, width, height);
          XFree(image);
-         if (!g_owncolmap)
+         if (tdata != data)
                  xfree(tdata);
-         return (HBITMAP) bitmap;
+         return (RD_HBITMAP) bitmap;
+ }
+ void
+ update_bitmap(int x, int y, int width, int height, uint8 * data)
+ {
+         int size;
+         uint8 *out;
+         uint8 *end;
+         uint8 *c_current;
+         uint8 *c_update;
+         uint8 *c_written;
+         int col;
+         int row;
+         BOOL matches_written;
+         BOOL matches_current;
+         size = width * height * 3;   /* force 24-bit color for libjpeg: R G B */
+         out = (uint8 *) xmalloc(size);
+         end = out + size;
+         switch (g_server_depth)
+         {
+                 case 24:
+                         translate24to24(data, out, end);
+                         break;
+                 case 16:
+                         translate16to24((uint16 *) data, out, end);
+                         break;
+                 case 15:
+                         translate15to24((uint16 *) data, out, end);
+                         break;
+                 case 8:
+                         translate8to24(data, out, end);
+                         break;
+         }
+         for (row=0; row<height; row++)
+         {
+                 /* truncate if out of bounds */
+                 if (g_height <= (y + row))
+                         break;
+                 c_update = out + row * width * 3;
+                 c_current = g_bitmap_data + (row + y) * g_width * 3 + x * 3;
+                 c_written = g_bitmap_data_last_write + (row + y) * g_width * 3 + x * 3;
+                 for (col = 0; col < width; col++)
+                 {
+                         /* truncate if out of bounds */
+                         if (g_width <= (x + col))
+                                 break;
+                         matches_written = True;
+                         matches_current = True;
+                         /* updates come in as BGR order... force RGB */
+                         if (*c_current != *(c_update+2))
+                         {
+                                 if (*(c_update+2) != *c_written)
+                                         matches_written = False;
+                                 matches_current = False;
+                                 *c_current = *(c_update+2);
+                         }
+                         c_written++;
+                         if (*(++c_current) != *(++c_update))
+                         {
+                                 if (*c_update != *c_written)
+                                         matches_written = False;
+                                 matches_current = False;
+                                 *c_current = *c_update;
+                         }
+                         c_written++;
+                         if (*(++c_current) != *(++c_update-2))
+                         {
+                                 if (*(c_update-2) != *c_written)
+                                         matches_written = False;
+                                 matches_current = False;
+                                 *c_current = *(c_update-2);
+                         }
+                         c_written++;
+                         c_current++;
+                         c_update++;
+                         if (!matches_current)
+                         {
+                                 if (matches_written)
+                                         g_pixels_changed--;
+                                 else
+                                         g_pixels_changed++;
+                         }
+                 }
+         }
+         //printf("g_pixels_changed=%d\n", g_pixels_changed);
+         xfree(out);
+         g_time_last_change = tod();
  }
  void
-Line 1256 
 ui_paint_bitmap(int x, int y, int cx, in
+Line 2803 
 ui_paint_bitmap(int x, int y, int cx, in
  {
          XImage *image;
          uint8 *tdata;
+         int bitmap_pad;
+         update_bitmap(x, y, width, height, data);
+         if (g_server_depth == 8)
+         {
+                 bitmap_pad = 8;
+         }
+         else
+         {
+                 bitmap_pad = g_bpp;
+                 if (g_bpp == 24)
+                         bitmap_pad = 32;
+         }
          tdata = (g_owncolmap ? data : translate_image(width, height, data));
          image = XCreateImage(g_display, g_visual, g_depth, ZPixmap, 0,
-                              (char *) tdata, width, height, g_server_bpp == 8 ? 8 : g_bpp, 0);
+                              (char *) tdata, width, height, bitmap_pad, 0);
          if (g_ownbackstore)
          {
                  XPutImage(g_display, g_backstore, g_gc, image, 0, 0, x, y, cx, cy);
                  XCopyArea(g_display, g_backstore, g_wnd, g_gc, x, y, cx, cy, x, y);
+                 ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                         (g_display, g_backstore, sw->wnd, g_gc, x, y, cx, cy,
+                                          x - sw->xoffset, y - sw->yoffset));
          }
          else
          {
                  XPutImage(g_display, g_wnd, g_gc, image, 0, 0, x, y, cx, cy);
+                 ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                         (g_display, g_wnd, sw->wnd, g_gc, x, y, cx, cy,
+                                          x - sw->xoffset, y - sw->yoffset));
          }
          XFree(image);
-         if (!g_owncolmap)
+         if (tdata != data)
                  xfree(tdata);
  }
  void
- ui_destroy_bitmap(HBITMAP bmp)
+ ui_destroy_bitmap(RD_HBITMAP bmp)
  {
          XFreePixmap(g_display, (Pixmap) bmp);
  }
- HGLYPH
+ RD_HGLYPH
  ui_create_glyph(int width, int height, uint8 * data)
  {
          XImage *image;
          Pixmap bitmap;
          int scanline;
-         GC gc;
          scanline = (width + 7) / 8;
          bitmap = XCreatePixmap(g_display, g_wnd, width, height, 1);
-         gc = XCreateGC(g_display, bitmap, 0, NULL);
+         if (g_create_glyph_gc == 0)
+                 g_create_glyph_gc = XCreateGC(g_display, bitmap, 0, NULL);
          image = XCreateImage(g_display, g_visual, 1, ZPixmap, 0, (char *) data,
                               width, height, 8, scanline);
-Line 1300 
 ui_create_glyph(int width, int height, u
+Line 2869 
 ui_create_glyph(int width, int height, u
          image->bitmap_bit_order = MSBFirst;
          XInitImage(image);
-         XPutImage(g_display, bitmap, gc, image, 0, 0, 0, 0, width, height);
+         XPutImage(g_display, bitmap, g_create_glyph_gc, image, 0, 0, 0, 0, width, height);
          XFree(image);
-         XFreeGC(g_display, gc);
+         return (RD_HGLYPH) bitmap;
-         return (HGLYPH) bitmap;
  }
  void
- ui_destroy_glyph(HGLYPH glyph)
+ ui_destroy_glyph(RD_HGLYPH glyph)
  {
          XFreePixmap(g_display, (Pixmap) glyph);
  }
- HCURSOR
+ RD_HCURSOR
  ui_create_cursor(unsigned int x, unsigned int y, int width, int height,
                   uint8 * andmask, uint8 * xormask)
  {
-         HGLYPH maskglyph, cursorglyph;
+         RD_HGLYPH maskglyph, cursorglyph;
          XColor bg, fg;
          Cursor xcursor;
          uint8 *cursor, *pcursor;
-Line 1381 
 ui_create_cursor(unsigned int x, unsigne
+Line 2949 
 ui_create_cursor(unsigned int x, unsigne
          ui_destroy_glyph(cursorglyph);
          xfree(mask);
          xfree(cursor);
-         return (HCURSOR) xcursor;
+         return (RD_HCURSOR) xcursor;
  }
  void
- ui_set_cursor(HCURSOR cursor)
+ ui_set_cursor(RD_HCURSOR cursor)
  {
          g_current_cursor = (Cursor) cursor;
          XDefineCursor(g_display, g_wnd, g_current_cursor);
+         ON_ALL_SEAMLESS_WINDOWS(XDefineCursor, (g_display, sw->wnd, g_current_cursor));
  }
  void
- ui_destroy_cursor(HCURSOR cursor)
+ ui_destroy_cursor(RD_HCURSOR cursor)
  {
          XFreeCursor(g_display, (Cursor) cursor);
  }
+ void
+ ui_set_null_cursor(void)
+ {
+         ui_set_cursor(g_null_cursor);
+ }
  #define MAKE_XCOLOR(xc,c) \
                  (xc)->red   = ((c)->red   << 8) | (c)->red; \
                  (xc)->green = ((c)->green << 8) | (c)->green; \
-Line 1404 
 ui_destroy_cursor(HCURSOR cursor)
+Line 2979 
 ui_destroy_cursor(HCURSOR cursor)
                  (xc)->flags = DoRed | DoGreen | DoBlue;
- HCOLOURMAP
+ RD_HCOLOURMAP
  ui_create_colourmap(COLOURMAP * colours)
  {
          COLOURENTRY *entry;
-Line 1478 
 ui_create_colourmap(COLOURMAP * colours)
+Line 3053 
 ui_create_colourmap(COLOURMAP * colours)
                          }
+                         map[i] = colour;
-                         /* byte swap here to make translate_image faster */
-                         map[i] = translate_colour(colour);
                  }
                  return map;
          }
-Line 1502 
 ui_create_colourmap(COLOURMAP * colours)
+Line 3075 
 ui_create_colourmap(COLOURMAP * colours)
                  XStoreColors(g_display, map, xcolours, ncolours);
                  xfree(xcolours);
-                 return (HCOLOURMAP) map;
+                 return (RD_HCOLOURMAP) map;
          }
  }
  void
- ui_destroy_colourmap(HCOLOURMAP map)
+ ui_destroy_colourmap(RD_HCOLOURMAP map)
  {
          if (!g_owncolmap)
                  xfree(map);
-Line 1516 
 ui_destroy_colourmap(HCOLOURMAP map)
+Line 3089 
 ui_destroy_colourmap(HCOLOURMAP map)
  }
  void
- ui_set_colourmap(HCOLOURMAP map)
+ ui_set_colourmap(RD_HCOLOURMAP map)
  {
          if (!g_owncolmap)
          {
-Line 1526 
 ui_set_colourmap(HCOLOURMAP map)
+Line 3099 
 ui_set_colourmap(HCOLOURMAP map)
                  g_colmap = (uint32 *) map;
          }
          else
+         {
                  XSetWindowColormap(g_display, g_wnd, (Colormap) map);
+                 ON_ALL_SEAMLESS_WINDOWS(XSetWindowColormap, (g_display, sw->wnd, (Colormap) map));
+         }
  }
  void
  ui_set_clip(int x, int y, int cx, int cy)
  {
-         XRectangle rect;
+         g_clip_rectangle.x = x;
+         g_clip_rectangle.y = y;
-         rect.x = x;
+         g_clip_rectangle.width = cx;
-         rect.y = y;
+         g_clip_rectangle.height = cy;
-         rect.width = cx;
+         XSetClipRectangles(g_display, g_gc, 0, 0, &g_clip_rectangle, 1, YXBanded);
-         rect.height = cy;
-         XSetClipRectangles(g_display, g_gc, 0, 0, &rect, 1, YXBanded);
  }
  void
  ui_reset_clip(void)
  {
-         XRectangle rect;
+         g_clip_rectangle.x = 0;
+         g_clip_rectangle.y = 0;
-         rect.x = 0;
+         g_clip_rectangle.width = g_width;
-         rect.y = 0;
+         g_clip_rectangle.height = g_height;
-         rect.width = g_width;
+         XSetClipRectangles(g_display, g_gc, 0, 0, &g_clip_rectangle, 1, YXBanded);
-         rect.height = g_height;
-         XSetClipRectangles(g_display, g_gc, 0, 0, &rect, 1, YXBanded);
  }
  void
-Line 1591 
 ui_patblt(uint8 opcode,
+Line 3163 
 ui_patblt(uint8 opcode,
          {
                  case 0: /* Solid */
                          SET_FOREGROUND(fgcolour);
-                         FILL_RECTANGLE(x, y, cx, cy);
+                         FILL_RECTANGLE_BACKSTORE(x, y, cx, cy);
                          break;
                  case 2: /* Hatch */
-Line 1602 
 ui_patblt(uint8 opcode,
+Line 3174 
 ui_patblt(uint8 opcode,
                          XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
                          XSetStipple(g_display, g_gc, fill);
                          XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
-                         FILL_RECTANGLE(x, y, cx, cy);
+                         FILL_RECTANGLE_BACKSTORE(x, y, cx, cy);
                          XSetFillStyle(g_display, g_gc, FillSolid);
                          XSetTSOrigin(g_display, g_gc, 0, 0);
-                         ui_destroy_glyph((HGLYPH) fill);
+                         ui_destroy_glyph((RD_HGLYPH) fill);
                          break;
                  case 3: /* Pattern */
-                         for (i = 0; i != 8; i++)
+                         if (brush->bd == 0)     /* rdp4 brush */
-                                 ipattern[7 - i] = brush->pattern[i];
+                         {
-                         fill = (Pixmap) ui_create_glyph(8, 8, ipattern);
+                                 for (i = 0; i != 8; i++)
+                                         ipattern[7 - i] = brush->pattern[i];
-                         SET_FOREGROUND(bgcolour);
+                                 fill = (Pixmap) ui_create_glyph(8, 8, ipattern);
-                         SET_BACKGROUND(fgcolour);
+                                 SET_FOREGROUND(bgcolour);
-                         XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 SET_BACKGROUND(fgcolour);
-                         XSetStipple(g_display, g_gc, fill);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
-                         XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
-                         FILL_RECTANGLE(x, y, cx, cy);
+                                 FILL_RECTANGLE_BACKSTORE(x, y, cx, cy);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
-                         XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
-                         XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
-                         ui_destroy_glyph((HGLYPH) fill);
+                         }
+                         else if (brush->bd->colour_code > 1)    /* > 1 bpp */
+                         {
+                                 fill = (Pixmap) ui_create_bitmap(8, 8, brush->bd->data);
+                                 XSetFillStyle(g_display, g_gc, FillTiled);
+                                 XSetTile(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 FILL_RECTANGLE_BACKSTORE(x, y, cx, cy);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_bitmap((RD_HBITMAP) fill);
+                         }
+                         else
+                         {
+                                 fill = (Pixmap) ui_create_glyph(8, 8, brush->bd->data);
+                                 SET_FOREGROUND(bgcolour);
+                                 SET_BACKGROUND(fgcolour);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 FILL_RECTANGLE_BACKSTORE(x, y, cx, cy);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
+                         }
                          break;
                  default:
-Line 1631 
 ui_patblt(uint8 opcode,
+Line 3227 
 ui_patblt(uint8 opcode,
          }
          RESET_FUNCTION(opcode);
+         if (g_ownbackstore)
+                 XCopyArea(g_display, g_backstore, g_wnd, g_gc, x, y, cx, cy, x, y);
+         ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                 (g_display, g_ownbackstore ? g_backstore : g_wnd, sw->wnd, g_gc,
+                                  x, y, cx, cy, x - sw->xoffset, y - sw->yoffset));
  }
  void
-Line 1639 
 ui_screenblt(uint8 opcode,
+Line 3241 
 ui_screenblt(uint8 opcode,
               /* src */ int srcx, int srcy)
  {
          SET_FUNCTION(opcode);
-         XCopyArea(g_display, g_wnd, g_wnd, g_gc, srcx, srcy, cx, cy, x, y);
          if (g_ownbackstore)
+         {
+                 XCopyArea(g_display, g_Unobscured ? g_wnd : g_backstore,
+                           g_wnd, g_gc, srcx, srcy, cx, cy, x, y);
                  XCopyArea(g_display, g_backstore, g_backstore, g_gc, srcx, srcy, cx, cy, x, y);
+         }
+         else
+         {
+                 XCopyArea(g_display, g_wnd, g_wnd, g_gc, srcx, srcy, cx, cy, x, y);
+         }
+         ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                 (g_display, g_ownbackstore ? g_backstore : g_wnd,
+                                  sw->wnd, g_gc, x, y, cx, cy, x - sw->xoffset, y - sw->yoffset));
          RESET_FUNCTION(opcode);
  }
  void
  ui_memblt(uint8 opcode,
            /* dest */ int x, int y, int cx, int cy,
-           /* src */ HBITMAP src, int srcx, int srcy)
+           /* src */ RD_HBITMAP src, int srcx, int srcy)
  {
          SET_FUNCTION(opcode);
          XCopyArea(g_display, (Pixmap) src, g_wnd, g_gc, srcx, srcy, cx, cy, x, y);
+         ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                 (g_display, (Pixmap) src, sw->wnd, g_gc,
+                                  srcx, srcy, cx, cy, x - sw->xoffset, y - sw->yoffset));
          if (g_ownbackstore)
                  XCopyArea(g_display, (Pixmap) src, g_backstore, g_gc, srcx, srcy, cx, cy, x, y);
          RESET_FUNCTION(opcode);
-Line 1660 
 ui_memblt(uint8 opcode,
+Line 3277 
 ui_memblt(uint8 opcode,
  void
  ui_triblt(uint8 opcode,
            /* dest */ int x, int y, int cx, int cy,
-           /* src */ HBITMAP src, int srcx, int srcy,
+           /* src */ RD_HBITMAP src, int srcx, int srcy,
            /* brush */ BRUSH * brush, int bgcolour, int fgcolour)
  {
          /* This is potentially difficult to do in general. Until someone
-Line 1698 
 ui_line(uint8 opcode,
+Line 3315 
 ui_line(uint8 opcode,
          SET_FUNCTION(opcode);
          SET_FOREGROUND(pen->colour);
          XDrawLine(g_display, g_wnd, g_gc, startx, starty, endx, endy);
+         ON_ALL_SEAMLESS_WINDOWS(XDrawLine, (g_display, sw->wnd, g_gc,
+                                             startx - sw->xoffset, starty - sw->yoffset,
+                                             endx - sw->xoffset, endy - sw->yoffset));
          if (g_ownbackstore)
                  XDrawLine(g_display, g_backstore, g_gc, startx, starty, endx, endy);
          RESET_FUNCTION(opcode);
-Line 1712 
 ui_rect(
+Line 3332 
 ui_rect(
          FILL_RECTANGLE(x, y, cx, cy);
  }
+ void
+ ui_polygon(uint8 opcode,
+            /* mode */ uint8 fillmode,
+            /* dest */ RD_POINT * point, int npoints,
+            /* brush */ BRUSH * brush, int bgcolour, int fgcolour)
+ {
+         uint8 style, i, ipattern[8];
+         Pixmap fill;
+         SET_FUNCTION(opcode);
+         switch (fillmode)
+         {
+                 case ALTERNATE:
+                         XSetFillRule(g_display, g_gc, EvenOddRule);
+                         break;
+                 case WINDING:
+                         XSetFillRule(g_display, g_gc, WindingRule);
+                         break;
+                 default:
+                         unimpl("fill mode %d\n", fillmode);
+         }
+         if (brush)
+                 style = brush->style;
+         else
+                 style = 0;
+         switch (style)
+         {
+                 case 0: /* Solid */
+                         SET_FOREGROUND(fgcolour);
+                         FILL_POLYGON((XPoint *) point, npoints);
+                         break;
+                 case 2: /* Hatch */
+                         fill = (Pixmap) ui_create_glyph(8, 8,
+                                                         hatch_patterns + brush->pattern[0] * 8);
+                         SET_FOREGROUND(fgcolour);
+                         SET_BACKGROUND(bgcolour);
+                         XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                         XSetStipple(g_display, g_gc, fill);
+                         XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                         FILL_POLYGON((XPoint *) point, npoints);
+                         XSetFillStyle(g_display, g_gc, FillSolid);
+                         XSetTSOrigin(g_display, g_gc, 0, 0);
+                         ui_destroy_glyph((RD_HGLYPH) fill);
+                         break;
+                 case 3: /* Pattern */
+                         if (brush->bd == 0)     /* rdp4 brush */
+                         {
+                                 for (i = 0; i != 8; i++)
+                                         ipattern[7 - i] = brush->pattern[i];
+                                 fill = (Pixmap) ui_create_glyph(8, 8, ipattern);
+                                 SET_FOREGROUND(bgcolour);
+                                 SET_BACKGROUND(fgcolour);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 FILL_POLYGON((XPoint *) point, npoints);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
+                         }
+                         else if (brush->bd->colour_code > 1)    /* > 1 bpp */
+                         {
+                                 fill = (Pixmap) ui_create_bitmap(8, 8, brush->bd->data);
+                                 XSetFillStyle(g_display, g_gc, FillTiled);
+                                 XSetTile(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 FILL_POLYGON((XPoint *) point, npoints);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_bitmap((RD_HBITMAP) fill);
+                         }
+                         else
+                         {
+                                 fill = (Pixmap) ui_create_glyph(8, 8, brush->bd->data);
+                                 SET_FOREGROUND(bgcolour);
+                                 SET_BACKGROUND(fgcolour);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 FILL_POLYGON((XPoint *) point, npoints);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
+                         }
+                         break;
+                 default:
+                         unimpl("brush %d\n", brush->style);
+         }
+         RESET_FUNCTION(opcode);
+ }
+ void
+ ui_polyline(uint8 opcode,
+             /* dest */ RD_POINT * points, int npoints,
+             /* pen */ PEN * pen)
+ {
+         /* TODO: set join style */
+         SET_FUNCTION(opcode);
+         SET_FOREGROUND(pen->colour);
+         XDrawLines(g_display, g_wnd, g_gc, (XPoint *) points, npoints, CoordModePrevious);
+         if (g_ownbackstore)
+                 XDrawLines(g_display, g_backstore, g_gc, (XPoint *) points, npoints,
+                            CoordModePrevious);
+         ON_ALL_SEAMLESS_WINDOWS(seamless_XDrawLines,
+                                 (sw->wnd, (XPoint *) points, npoints, sw->xoffset, sw->yoffset));
+         RESET_FUNCTION(opcode);
+ }
+ void
+ ui_ellipse(uint8 opcode,
+            /* mode */ uint8 fillmode,
+            /* dest */ int x, int y, int cx, int cy,
+            /* brush */ BRUSH * brush, int bgcolour, int fgcolour)
+ {
+         uint8 style, i, ipattern[8];
+         Pixmap fill;
+         SET_FUNCTION(opcode);
+         if (brush)
+                 style = brush->style;
+         else
+                 style = 0;
+         switch (style)
+         {
+                 case 0: /* Solid */
+                         SET_FOREGROUND(fgcolour);
+                         DRAW_ELLIPSE(x, y, cx, cy, fillmode);
+                         break;
+                 case 2: /* Hatch */
+                         fill = (Pixmap) ui_create_glyph(8, 8,
+                                                         hatch_patterns + brush->pattern[0] * 8);
+                         SET_FOREGROUND(fgcolour);
+                         SET_BACKGROUND(bgcolour);
+                         XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                         XSetStipple(g_display, g_gc, fill);
+                         XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                         DRAW_ELLIPSE(x, y, cx, cy, fillmode);
+                         XSetFillStyle(g_display, g_gc, FillSolid);
+                         XSetTSOrigin(g_display, g_gc, 0, 0);
+                         ui_destroy_glyph((RD_HGLYPH) fill);
+                         break;
+                 case 3: /* Pattern */
+                         if (brush->bd == 0)     /* rdp4 brush */
+                         {
+                                 for (i = 0; i != 8; i++)
+                                         ipattern[7 - i] = brush->pattern[i];
+                                 fill = (Pixmap) ui_create_glyph(8, 8, ipattern);
+                                 SET_FOREGROUND(bgcolour);
+                                 SET_BACKGROUND(fgcolour);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 DRAW_ELLIPSE(x, y, cx, cy, fillmode);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
+                         }
+                         else if (brush->bd->colour_code > 1)    /* > 1 bpp */
+                         {
+                                 fill = (Pixmap) ui_create_bitmap(8, 8, brush->bd->data);
+                                 XSetFillStyle(g_display, g_gc, FillTiled);
+                                 XSetTile(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 DRAW_ELLIPSE(x, y, cx, cy, fillmode);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_bitmap((RD_HBITMAP) fill);
+                         }
+                         else
+                         {
+                                 fill = (Pixmap) ui_create_glyph(8, 8, brush->bd->data);
+                                 SET_FOREGROUND(bgcolour);
+                                 SET_BACKGROUND(fgcolour);
+                                 XSetFillStyle(g_display, g_gc, FillOpaqueStippled);
+                                 XSetStipple(g_display, g_gc, fill);
+                                 XSetTSOrigin(g_display, g_gc, brush->xorigin, brush->yorigin);
+                                 DRAW_ELLIPSE(x, y, cx, cy, fillmode);
+                                 XSetFillStyle(g_display, g_gc, FillSolid);
+                                 XSetTSOrigin(g_display, g_gc, 0, 0);
+                                 ui_destroy_glyph((RD_HGLYPH) fill);
+                         }
+                         break;
+                 default:
+                         unimpl("brush %d\n", brush->style);
+         }
+         RESET_FUNCTION(opcode);
+ }
  /* warning, this function only draws on wnd or backstore, not both */
  void
  ui_draw_glyph(int mixmode,
                /* dest */ int x, int y, int cx, int cy,
-               /* src */ HGLYPH glyph, int srcx, int srcy,
+               /* src */ RD_HGLYPH glyph, int srcx, int srcy,
                int bgcolour, int fgcolour)
  {
          SET_FOREGROUND(fgcolour);
-Line 1736 
 ui_draw_glyph(int mixmode,
+Line 3559 
 ui_draw_glyph(int mixmode,
  {\
    glyph = cache_get_font (font, ttext[idx]);\
    if (!(flags & TEXT2_IMPLICIT_X))\
+   {\
+     xyoffset = ttext[++idx];\
+     if ((xyoffset & 0x80))\
      {\
-       xyoffset = ttext[++idx];\
+       if (flags & TEXT2_VERTICAL)\
-       if ((xyoffset & 0x80))\
+         y += ttext[idx+1] | (ttext[idx+2] << 8);\
-         {\
-           if (flags & TEXT2_VERTICAL) \
-             y += ttext[idx+1] | (ttext[idx+2] << 8);\
-           else\
-             x += ttext[idx+1] | (ttext[idx+2] << 8);\
-           idx += 2;\
-         }\
        else\
-         {\
+         x += ttext[idx+1] | (ttext[idx+2] << 8);\
-           if (flags & TEXT2_VERTICAL) \
+       idx += 2;\
-             y += xyoffset;\
-           else\
-             x += xyoffset;\
-         }\
      }\
-   if (glyph != NULL)\
+     else\
      {\
-       ui_draw_glyph (mixmode, x + glyph->offset,\
+       if (flags & TEXT2_VERTICAL)\
-                      y + glyph->baseline,\
+         y += xyoffset;\
-                      glyph->width, glyph->height,\
+       else\
-                      glyph->pixmap, 0, 0, bgcolour, fgcolour);\
+         x += xyoffset;\
-       if (flags & TEXT2_IMPLICIT_X)\
-         x += glyph->width;\
      }\
+   }\
+   if (glyph != NULL)\
+   {\
+     x1 = x + glyph->offset;\
+     y1 = y + glyph->baseline;\
+     XSetStipple(g_display, g_gc, (Pixmap) glyph->pixmap);\
+     XSetTSOrigin(g_display, g_gc, x1, y1);\
+     FILL_RECTANGLE_BACKSTORE(x1, y1, glyph->width, glyph->height);\
+     if (flags & TEXT2_IMPLICIT_X)\
+       x += glyph->width;\
+   }\
  }
  void
- ui_draw_text(uint8 font, uint8 flags, int mixmode, int x, int y,
+ ui_draw_text(uint8 font, uint8 flags, uint8 opcode, int mixmode, int x, int y,
               int clipx, int clipy, int clipcx, int clipcy,
-              int boxx, int boxy, int boxcx, int boxcy, int bgcolour,
+              int boxx, int boxy, int boxcx, int boxcy, BRUSH * brush,
-              int fgcolour, uint8 * text, uint8 length)
+              int bgcolour, int fgcolour, uint8 * text, uint8 length)
  {
+         /* TODO: use brush appropriately */
          FONTGLYPH *glyph;
-         int i, j, xyoffset;
+         int i, j, xyoffset, x1, y1;
          DATABLOB *entry;
          SET_FOREGROUND(bgcolour);
+         /* Sometimes, the boxcx value is something really large, like
+. This makes XCopyArea fail with Xvnc. The code below
+            is a quick fix. */
+         if (boxx + boxcx > g_width)
+                 boxcx = g_width - boxx;
          if (boxcx > 1)
          {
                  FILL_RECTANGLE_BACKSTORE(boxx, boxy, boxcx, boxcy);
-Line 1786 
 ui_draw_text(uint8 font, uint8 flags, in
+Line 3618 
 ui_draw_text(uint8 font, uint8 flags, in
                  FILL_RECTANGLE_BACKSTORE(clipx, clipy, clipcx, clipcy);
          }
+         SET_FOREGROUND(fgcolour);
+         SET_BACKGROUND(bgcolour);
+         XSetFillStyle(g_display, g_gc, FillStippled);
          /* Paint text, character by character */
          for (i = 0; i < length;)
          {
                  switch (text[i])
                  {
                          case 0xff:
-                                 if (i + 2 < length)
+                                 /* At least two bytes needs to follow */
-                                         cache_put_text(text[i + 1], text, text[i + 2]);
+                                 if (i + 3 > length)
-                                 else
                                  {
-                                         error("this shouldn't be happening\n");
+                                         warning("Skipping short 0xff command:");
-                                         exit(1);
+                                         for (j = 0; j < length; j++)
+                                                 fprintf(stderr, "%02x ", text[j]);
+                                         fprintf(stderr, "\n");
+                                         i = length = 0;
+                                         break;
                                  }
+                                 cache_put_text(text[i + 1], text, text[i + 2]);
+                                 i += 3;
+                                 length -= i;
                                  /* this will move pointer from start to first character after FF command */
-                                 length -= i + 3;
+                                 text = &(text[i]);
-                                 text = &(text[i + 3]);
                                  i = 0;
                                  break;
                          case 0xfe:
+                                 /* At least one byte needs to follow */
+                                 if (i + 2 > length)
+                                 {
+                                         warning("Skipping short 0xfe command:");
+                                         for (j = 0; j < length; j++)
+                                                 fprintf(stderr, "%02x ", text[j]);
+                                         fprintf(stderr, "\n");
+                                         i = length = 0;
+                                         break;
+                                 }
                                  entry = cache_get_text(text[i + 1]);
-                                 if (entry != NULL)
+                                 if (entry->data != NULL)
                                  {
-                                         if ((((uint8 *) (entry->data))[1] ==
+                                         if ((((uint8 *) (entry->data))[1] == 0)
-) && (!(flags & TEXT2_IMPLICIT_X)))
+                                             && (!(flags & TEXT2_IMPLICIT_X)) && (i + 2 < length))
                                          {
                                                  if (flags & TEXT2_VERTICAL)
                                                          y += text[i + 2];
-Line 1836 
 ui_draw_text(uint8 font, uint8 flags, in
+Line 3687 
 ui_draw_text(uint8 font, uint8 flags, in
                                  break;
                  }
          }
+         XSetFillStyle(g_display, g_gc, FillSolid);
          if (g_ownbackstore)
          {
                  if (boxcx > 1)
+                 {
                          XCopyArea(g_display, g_backstore, g_wnd, g_gc, boxx,
                                    boxy, boxcx, boxcy, boxx, boxy);
+                         ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                                 (g_display, g_backstore, sw->wnd, g_gc,
+                                                  boxx, boxy,
+                                                  boxcx, boxcy,
+                                                  boxx - sw->xoffset, boxy - sw->yoffset));
+                 }
                  else
+                 {
                          XCopyArea(g_display, g_backstore, g_wnd, g_gc, clipx,
                                    clipy, clipcx, clipcy, clipx, clipy);
+                         ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                                 (g_display, g_backstore, sw->wnd, g_gc,
+                                                  clipx, clipy,
+                                                  clipcx, clipcy, clipx - sw->xoffset,
+                                                  clipy - sw->yoffset));
+                 }
          }
  }
-Line 1856 
 ui_desktop_save(uint32 offset, int x, in
+Line 3724 
 ui_desktop_save(uint32 offset, int x, in
          if (g_ownbackstore)
          {
                  image = XGetImage(g_display, g_backstore, x, y, cx, cy, AllPlanes, ZPixmap);
+                 exit_if_null(image);
          }
          else
          {
                  pix = XCreatePixmap(g_display, g_wnd, cx, cy, g_depth);
                  XCopyArea(g_display, g_wnd, pix, g_gc, x, y, cx, cy, 0, 0);
                  image = XGetImage(g_display, pix, 0, 0, cx, cy, AllPlanes, ZPixmap);
+                 exit_if_null(image);
                  XFreePixmap(g_display, pix);
          }
-Line 1883 
 ui_desktop_restore(uint32 offset, int x,
+Line 3753 
 ui_desktop_restore(uint32 offset, int x,
                  return;
          image = XCreateImage(g_display, g_visual, g_depth, ZPixmap, 0,
-                              (char *) data, cx, cy, BitmapPad(g_display), cx * g_bpp / 8);
+                              (char *) data, cx, cy, g_bpp, 0);
          if (g_ownbackstore)
          {
                  XPutImage(g_display, g_backstore, g_gc, image, 0, 0, x, y, cx, cy);
                  XCopyArea(g_display, g_backstore, g_wnd, g_gc, x, y, cx, cy, x, y);
+                 ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                         (g_display, g_backstore, sw->wnd, g_gc,
+                                          x, y, cx, cy, x - sw->xoffset, y - sw->yoffset));
          }
          else
          {
                  XPutImage(g_display, g_wnd, g_gc, image, 0, 0, x, y, cx, cy);
+                 ON_ALL_SEAMLESS_WINDOWS(XCopyArea,
+                                         (g_display, g_wnd, sw->wnd, g_gc, x, y, cx, cy,
+                                          x - sw->xoffset, y - sw->yoffset));
          }
          XFree(image);
  }
+ /* these do nothing here but are used in uiports */
+ void
+ ui_begin_update(void)
+ {
+ }
+ void
+ ui_end_update(void)
+ {
+ }
+ void
+ ui_seamless_begin(RD_BOOL hidden)
+ {
+         if (!g_seamless_rdp)
+                 return;
+         if (g_seamless_started)
+                 return;
+         g_seamless_started = True;
+         g_seamless_hidden = hidden;
+         if (!hidden)
+                 ui_seamless_toggle();
+ }
+ void
+ ui_seamless_hide_desktop()
+ {
+         if (!g_seamless_rdp)
+                 return;
+         if (!g_seamless_started)
+                 return;
+         if (g_seamless_active)
+                 ui_seamless_toggle();
+         g_seamless_hidden = True;
+ }
+ void
+ ui_seamless_unhide_desktop()
+ {
+         if (!g_seamless_rdp)
+                 return;
+         if (!g_seamless_started)
+                 return;
+         g_seamless_hidden = False;
+         ui_seamless_toggle();
+ }
+ void
+ ui_seamless_toggle()
+ {
+         if (!g_seamless_rdp)
+                 return;
+         if (!g_seamless_started)
+                 return;
+         if (g_seamless_hidden)
+                 return;
+         if (g_seamless_active)
+         {
+                 /* Deactivate */
+                 while (g_seamless_windows)
+                 {
+                         XDestroyWindow(g_display, g_seamless_windows->wnd);
+                         sw_remove_window(g_seamless_windows);
+                 }
+                 XMapWindow(g_display, g_wnd);
+         }
+         else
+         {
+                 /* Activate */
+                 XUnmapWindow(g_display, g_wnd);
+                 seamless_send_sync();
+         }
+         g_seamless_active = !g_seamless_active;
+ }
+ void
+ ui_seamless_create_window(unsigned long id, unsigned long group, unsigned long parent,
+                           unsigned long flags)
+ {
+         Window wnd;
+         XSetWindowAttributes attribs;
+         XClassHint *classhints;
+         XSizeHints *sizehints;
+         XWMHints *wmhints;
+         long input_mask;
+         seamless_window *sw, *sw_parent;
+         if (!g_seamless_active)
+                 return;
+         /* Ignore CREATEs for existing windows */
+         sw = sw_get_window_by_id(id);
+         if (sw)
+                 return;
+         get_window_attribs(&attribs);
+         wnd = XCreateWindow(g_display, RootWindowOfScreen(g_screen), -1, -1, 1, 1, 0, g_depth,
+                             InputOutput, g_visual,
+                             CWBackPixel | CWBackingStore | CWColormap | CWBorderPixel, &attribs);
+         XStoreName(g_display, wnd, "SeamlessRDP");
+         ewmh_set_wm_name(wnd, "SeamlessRDP");
+         mwm_hide_decorations(wnd);
+         classhints = XAllocClassHint();
+         if (classhints != NULL)
+         {
+                 classhints->res_name = "rdesktop";
+                 classhints->res_class = "SeamlessRDP";
+                 XSetClassHint(g_display, wnd, classhints);
+                 XFree(classhints);
+         }
+         /* WM_NORMAL_HINTS */
+         sizehints = XAllocSizeHints();
+         if (sizehints != NULL)
+         {
+                 sizehints->flags = USPosition;
+                 XSetWMNormalHints(g_display, wnd, sizehints);
+                 XFree(sizehints);
+         }
+         /* Parent-less transient windows */
+         if (parent == 0xFFFFFFFF)
+         {
+                 XSetTransientForHint(g_display, wnd, RootWindowOfScreen(g_screen));
+                 /* Some buggy wm:s (kwin) do not handle the above, so fake it
+                    using some other hints. */
+                 ewmh_set_window_popup(wnd);
+         }
+         /* Normal transient windows */
+         else if (parent != 0x00000000)
+         {
+                 sw_parent = sw_get_window_by_id(parent);
+                 if (sw_parent)
+                         XSetTransientForHint(g_display, wnd, sw_parent->wnd);
+                 else
+                         warning("ui_seamless_create_window: No parent window 0x%lx\n", parent);
+         }
+         if (flags & SEAMLESSRDP_CREATE_MODAL)
+         {
+                 /* We do this to support buggy wm:s (*cough* metacity *cough*)
+                    somewhat at least */
+                 if (parent == 0x00000000)
+                         XSetTransientForHint(g_display, wnd, RootWindowOfScreen(g_screen));
+                 ewmh_set_window_modal(wnd);
+         }
+         if (flags & SEAMLESSRDP_CREATE_TOPMOST)
+         {
+                 /* Make window always-on-top */
+                 ewmh_set_window_above(wnd);
+         }
+         /* FIXME: Support for Input Context:s */
+         get_input_mask(&input_mask);
+         input_mask |= PropertyChangeMask;
+         XSelectInput(g_display, wnd, input_mask);
+         /* handle the WM_DELETE_WINDOW protocol. */
+         XSetWMProtocols(g_display, wnd, &g_kill_atom, 1);
+         sw = xmalloc(sizeof(seamless_window));
+         memset(sw, 0, sizeof(seamless_window));
+         sw->wnd = wnd;
+         sw->id = id;
+         sw->group = sw_find_group(group, False);
+         sw->group->refcnt++;
+         sw->state = SEAMLESSRDP_NOTYETMAPPED;
+         sw->desktop = 0;
+         sw->position_timer = xmalloc(sizeof(struct timeval));
+         timerclear(sw->position_timer);
+         sw->outstanding_position = False;
+         sw->outpos_serial = 0;
+         sw->outpos_xoffset = sw->outpos_yoffset = 0;
+         sw->outpos_width = sw->outpos_height = 0;
+         sw->next = g_seamless_windows;
+         g_seamless_windows = sw;
+         /* WM_HINTS */
+         wmhints = XAllocWMHints();
+         if (wmhints)
+         {
+                 wmhints->flags = WindowGroupHint;
+                 wmhints->window_group = sw->group->wnd;
+                 XSetWMHints(g_display, sw->wnd, wmhints);
+                 XFree(wmhints);
+         }
+ }
+ void
+ ui_seamless_destroy_window(unsigned long id, unsigned long flags)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_destroy_window: No information for window 0x%lx\n", id);
+                 return;
+         }
+         XDestroyWindow(g_display, sw->wnd);
+         sw_remove_window(sw);
+ }
+ void
+ ui_seamless_destroy_group(unsigned long id, unsigned long flags)
+ {
+         seamless_window *sw, *sw_next;
+         if (!g_seamless_active)
+                 return;
+         for (sw = g_seamless_windows; sw; sw = sw_next)
+         {
+                 sw_next = sw->next;
+                 if (sw->group->id == id)
+                 {
+                         XDestroyWindow(g_display, sw->wnd);
+                         sw_remove_window(sw);
+                 }
+         }
+ }
+ void
+ ui_seamless_seticon(unsigned long id, const char *format, int width, int height, int chunk,
+                     const char *data, int chunk_len)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_seticon: No information for window 0x%lx\n", id);
+                 return;
+         }
+         if (chunk == 0)
+         {
+                 if (sw->icon_size)
+                         warning("ui_seamless_seticon: New icon started before previous completed\n");
+                 if (strcmp(format, "RGBA") != 0)
+                 {
+                         warning("ui_seamless_seticon: Uknown icon format \"%s\"\n", format);
+                         return;
+                 }
+                 sw->icon_size = width * height * 4;
+                 if (sw->icon_size > 32 * 32 * 4)
+                 {
+                         warning("ui_seamless_seticon: Icon too large (%d bytes)\n", sw->icon_size);
+                         sw->icon_size = 0;
+                         return;
+                 }
+                 sw->icon_offset = 0;
+         }
+         else
+         {
+                 if (!sw->icon_size)
+                         return;
+         }
+         if (chunk_len > (sw->icon_size - sw->icon_offset))
+         {
+                 warning("ui_seamless_seticon: Too large chunk received (%d bytes > %d bytes)\n",
+                         chunk_len, sw->icon_size - sw->icon_offset);
+                 sw->icon_size = 0;
+                 return;
+         }
+         memcpy(sw->icon_buffer + sw->icon_offset, data, chunk_len);
+         sw->icon_offset += chunk_len;
+         if (sw->icon_offset == sw->icon_size)
+         {
+                 ewmh_set_icon(sw->wnd, width, height, sw->icon_buffer);
+                 sw->icon_size = 0;
+         }
+ }
+ void
+ ui_seamless_delicon(unsigned long id, const char *format, int width, int height)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_seticon: No information for window 0x%lx\n", id);
+                 return;
+         }
+         if (strcmp(format, "RGBA") != 0)
+         {
+                 warning("ui_seamless_seticon: Uknown icon format \"%s\"\n", format);
+                 return;
+         }
+         ewmh_del_icon(sw->wnd, width, height);
+ }
+ void
+ ui_seamless_move_window(unsigned long id, int x, int y, int width, int height, unsigned long flags)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_move_window: No information for window 0x%lx\n", id);
+                 return;
+         }
+         /* We ignore server updates until it has handled our request. */
+         if (sw->outstanding_position)
+                 return;
+         if (!width || !height)
+                 /* X11 windows must be at least 1x1 */
+                 return;
+         sw->xoffset = x;
+         sw->yoffset = y;
+         sw->width = width;
+         sw->height = height;
+         /* If we move the window in a maximized state, then KDE won't
+            accept restoration */
+         switch (sw->state)
+         {
+                 case SEAMLESSRDP_MINIMIZED:
+                 case SEAMLESSRDP_MAXIMIZED:
+                         return;
+         }
+         /* FIXME: Perhaps use ewmh_net_moveresize_window instead */
+         XMoveResizeWindow(g_display, sw->wnd, sw->xoffset, sw->yoffset, sw->width, sw->height);
+ }
+ void
+ ui_seamless_restack_window(unsigned long id, unsigned long behind, unsigned long flags)
+ {
+         seamless_window *sw;
+         XWindowChanges values;
+         unsigned long restack_serial;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_restack_window: No information for window 0x%lx\n", id);
+                 return;
+         }
+         if (behind)
+         {
+                 seamless_window *sw_behind;
+                 sw_behind = sw_get_window_by_id(behind);
+                 if (!sw_behind)
+                 {
+                         warning("ui_seamless_restack_window: No information for behind window 0x%lx\n", behind);
+                         return;
+                 }
+                 if (!g_seamless_broken_restack)
+                 {
+                         values.stack_mode = Below;
+                         values.sibling = sw_behind->wnd;
+                         restack_serial = XNextRequest(g_display);
+                         XReconfigureWMWindow(g_display, sw->wnd, DefaultScreen(g_display),
+                                              CWStackMode | CWSibling, &values);
+                         sw_wait_configurenotify(sw->wnd, restack_serial);
+                 }
+         }
+         else
+         {
+                 values.stack_mode = Above;
+                 restack_serial = XNextRequest(g_display);
+                 XReconfigureWMWindow(g_display, sw->wnd, DefaultScreen(g_display), CWStackMode,
+                                      &values);
+                 sw_wait_configurenotify(sw->wnd, restack_serial);
+         }
+         sw_restack_window(sw, behind);
+         if (flags & SEAMLESSRDP_CREATE_TOPMOST)
+         {
+                 /* Make window always-on-top */
+                 ewmh_set_window_above(sw->wnd);
+         }
+ }
+ void
+ ui_seamless_settitle(unsigned long id, const char *title, unsigned long flags)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_settitle: No information for window 0x%lx\n", id);
+                 return;
+         }
+         /* FIXME: Might want to convert the name for non-EWMH WMs */
+         XStoreName(g_display, sw->wnd, title);
+         ewmh_set_wm_name(sw->wnd, title);
+ }
+ void
+ ui_seamless_setstate(unsigned long id, unsigned int state, unsigned long flags)
+ {
+         seamless_window *sw;
+         if (!g_seamless_active)
+                 return;
+         sw = sw_get_window_by_id(id);
+         if (!sw)
+         {
+                 warning("ui_seamless_setstate: No information for window 0x%lx\n", id);
+                 return;
+         }
+         switch (state)
+         {
+                 case SEAMLESSRDP_NORMAL:
+                 case SEAMLESSRDP_MAXIMIZED:
+                         ewmh_change_state(sw->wnd, state);
+                         XMapWindow(g_display, sw->wnd);
+                         break;
+                 case SEAMLESSRDP_MINIMIZED:
+                         /* EWMH says: "if an Application asks to toggle _NET_WM_STATE_HIDDEN
+                            the Window Manager should probably just ignore the request, since
+                            _NET_WM_STATE_HIDDEN is a function of some other aspect of the window
+                            such as minimization, rather than an independent state." Besides,
+                            XIconifyWindow is easier. */
+                         if (sw->state == SEAMLESSRDP_NOTYETMAPPED)
+                         {
+                                 XWMHints *hints;
+                                 hints = XGetWMHints(g_display, sw->wnd);
+                                 if (hints)
+                                 {
+                                         hints->flags |= StateHint;
+                                         hints->initial_state = IconicState;
+                                         XSetWMHints(g_display, sw->wnd, hints);
+                                         XFree(hints);
+                                 }
+                                 XMapWindow(g_display, sw->wnd);
+                         }
+                         else
+                                 XIconifyWindow(g_display, sw->wnd, DefaultScreen(g_display));
+                         break;
+                 default:
+                         warning("SeamlessRDP: Invalid state %d\n", state);
+                         break;
+         }
+         sw->state = state;
+ }
+ void
+ ui_seamless_syncbegin(unsigned long flags)
+ {
+         if (!g_seamless_active)
+                 return;
+         /* Destroy all seamless windows */
+         while (g_seamless_windows)
+         {
+                 XDestroyWindow(g_display, g_seamless_windows->wnd);
+                 sw_remove_window(g_seamless_windows);
+         }
+ }
+ void
+ ui_seamless_ack(unsigned int serial)
+ {
+         seamless_window *sw;
+         for (sw = g_seamless_windows; sw; sw = sw->next)
+         {
+                 if (sw->outstanding_position && (sw->outpos_serial == serial))
+                 {
+                         sw->xoffset = sw->outpos_xoffset;
+                         sw->yoffset = sw->outpos_yoffset;
+                         sw->width = sw->outpos_width;
+                         sw->height = sw->outpos_height;
+                         sw->outstanding_position = False;
+                         /* Do a complete redraw of the window as part of the
+                            completion of the move. This is to remove any
+                            artifacts caused by our lack of synchronization. */
+                         XCopyArea(g_display, g_backstore,
+                                   sw->wnd, g_gc,
+                                   sw->xoffset, sw->yoffset, sw->width, sw->height, 0, 0);
+                         break;
+                 }
+         }
+ }

 Legend:



Removed from v.498
 


changed lines


 
Added in v.1508
 Legend:



Removed from v.498
 


changed lines


 
Added in v.1508
-Removed from v.498
+Added in v.1508

	ViewVC Help
Powered by ViewVC 1.1.26