--- sourceforge.net/trunk/rdesktop/xwin.c	2002/11/11 23:15:27	257
+++ sourceforge.net/trunk/rdesktop/xwin.c	2002/11/26 10:09:14	279
@@ -29,6 +29,7 @@
 extern BOOL sendmotion;
 extern BOOL fullscreen;
 extern BOOL grab_keyboard;
+extern BOOL hide_decorations;
 extern char title[];
 BOOL enable_compose = False;
 BOOL focused;
@@ -46,6 +47,7 @@
 static XIC IC;
 static XModifierKeymap *mod_map;
 static Cursor current_cursor;
+static Atom protocol_atom, kill_atom;
 
 /* endianness */
 static BOOL host_be;
@@ -55,6 +57,20 @@
 static BOOL ownbackstore;
 static Pixmap backstore;
 
+/* MWM decorations */
+#define MWM_HINTS_DECORATIONS   (1L << 1)
+#define PROP_MOTIF_WM_HINTS_ELEMENTS    5
+typedef struct
+{
+	unsigned long flags;
+	unsigned long functions;
+	unsigned long decorations;
+	long inputMode;
+	unsigned long status;
+}
+PropMotifWmHints;
+
+
 #define FILL_RECTANGLE(x,y,cx,cy)\
 { \
 	XFillRectangle(display, wnd, gc, x, y, cx, cy); \
@@ -63,11 +79,13 @@
 }
 
 /* colour maps */
+BOOL owncolmap = False;
 static Colormap xcolmap;
 static uint32 *colmap;
 
-#define SET_FOREGROUND(col)	XSetForeground(display, gc, translate_colour(colmap[col]));
-#define SET_BACKGROUND(col)	XSetBackground(display, gc, translate_colour(colmap[col]));
+#define TRANSLATE(col)		( owncolmap ? col : translate_colour(colmap[col]) )
+#define SET_FOREGROUND(col)	XSetForeground(display, gc, TRANSLATE(col));
+#define SET_BACKGROUND(col)	XSetBackground(display, gc, TRANSLATE(col));
 
 static int rop2_map[] = {
 	GXclear,		/* 0 */
@@ -91,6 +109,28 @@
 #define SET_FUNCTION(rop2)	{ if (rop2 != ROP2_COPY) XSetFunction(display, gc, rop2_map[rop2]); }
 #define RESET_FUNCTION(rop2)	{ if (rop2 != ROP2_COPY) XSetFunction(display, gc, GXcopy); }
 
+void
+mwm_hide_decorations(void)
+{
+	PropMotifWmHints motif_hints;
+	Atom hintsatom;
+
+	/* setup the property */
+	motif_hints.flags = MWM_HINTS_DECORATIONS;
+	motif_hints.decorations = 0;
+
+	/* get the atom for the property */
+	hintsatom = XInternAtom(display, "_MOTIF_WM_HINTS", False);
+	if (!hintsatom)
+	{
+		error("Failed to get atom _MOTIF_WM_HINTS\n");
+		return;
+	}
+
+	XChangeProperty(display, wnd, hintsatom, hintsatom, 32, PropModeReplace,
+			(unsigned char *) &motif_hints, PROP_MOTIF_WM_HINTS_ELEMENTS);
+}
+
 static void
 translate8(uint8 * data, uint8 * out, uint8 * end)
 {
@@ -251,7 +291,16 @@
 		return False;
 	}
 
-	xcolmap = DefaultColormapOfScreen(screen);
+	if (owncolmap != True)
+	{
+		xcolmap = DefaultColormapOfScreen(screen);
+		if (depth <= 8)
+		{
+			printf("You're using a screen depth of 8-bits or lower\n");
+			printf("If you get scewed colours, try the -C switch\n");
+		}
+	}
+
 	gc = XCreateGC(display, RootWindowOfScreen(screen), 0, NULL);
 
 	if (DoesBackingStore(screen) != Always)
@@ -261,6 +310,21 @@
 	host_be = !(BOOL) (*(uint8 *) (&test));
 	xserver_be = (ImageByteOrder(display) == MSBFirst);
 
+	if ((width == 0) || (height == 0))
+	{
+		/* Fetch geometry from _NET_WORKAREA */
+		uint32 xpos, ypos;
+
+		if (get_current_workarea(&xpos, &ypos, &width, &height) < 0)
+		{
+			error("Failed to get workarea.\n");
+			error("Perhaps your window manager does not support EWMH?\n");
+			error("Defaulting to geometry 800x600\n");
+			width = 800;
+			height = 600;
+		}
+	}
+
 	if (fullscreen)
 	{
 		width = WidthOfScreen(screen);
@@ -328,6 +392,9 @@
 
 	XStoreName(display, wnd, title);
 
+	if (hide_decorations)
+		mwm_hide_decorations();
+
 	classhints = XAllocClassHint();
 	if (classhints != NULL)
 	{
@@ -381,6 +448,11 @@
 	focused = False;
 	mouse_in_wnd = False;
 
+	/* handle the WM_DELETE_WINDOW protocol */
+	protocol_atom = XInternAtom(display, "WM_PROTOCOLS", True);
+	kill_atom = XInternAtom(display, "WM_DELETE_WINDOW", True);
+	XSetWMProtocols(display, wnd, &kill_atom, 1);
+
 	return True;
 }
 
@@ -418,8 +490,9 @@
 	}
 }
 
-/* Process all events in Xlib queue */
-static void
+/* Process all events in Xlib queue 
+   Returns 0 after user quit, 1 otherwise */
+static int
 xwin_process_events(void)
 {
 	XEvent xevent;
@@ -447,6 +520,14 @@
 
 		switch (xevent.type)
 		{
+			case ClientMessage:
+				/* the window manager told us to quit */
+				if ((xevent.xclient.message_type == protocol_atom)
+				    && (xevent.xclient.data.l[0] == kill_atom))
+					/* Quit */
+					return 0;
+				break;
+
 			case KeyPress:
 				if (IC != NULL)
 					/* Multi_key compatible version */
@@ -469,7 +550,8 @@
 						      str, sizeof(str), &keysym, NULL);
 				}
 
-				DEBUG_KBD(("KeyPress for (keysym 0x%lx, %s)\n", keysym, get_ksname(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))
@@ -528,7 +610,8 @@
 				if (xevent.xfocus.mode == NotifyGrab)
 					break;
 				focused = True;
-				XQueryPointer(display, wnd, &wdummy, &wdummy, &dummy, &dummy, &dummy, &dummy, &state);
+				XQueryPointer(display, wnd, &wdummy, &wdummy, &dummy, &dummy,
+					      &dummy, &dummy, &state);
 				reset_modifier_keys(state);
 				if (grab_keyboard && mouse_in_wnd)
 					XGrabKeyboard(display, wnd, True,
@@ -549,7 +632,8 @@
 				mouse_in_wnd = True;
 				if (fullscreen)
 				{
-					XSetInputFocus(display, wnd, RevertToPointerRoot, CurrentTime);
+					XSetInputFocus(display, wnd, RevertToPointerRoot,
+						       CurrentTime);
 					break;
 				}
 				if (focused)
@@ -587,9 +671,12 @@
 
 		}
 	}
+	/* Keep going */
+	return 1;
 }
 
-void
+/* Returns 0 after user quit, 1 otherwise */
+int
 ui_select(int rdp_socket)
 {
 	int n = (rdp_socket > x_socket) ? rdp_socket + 1 : x_socket + 1;
@@ -600,7 +687,9 @@
 	while (True)
 	{
 		/* Process any events already waiting */
-		xwin_process_events();
+		if (!xwin_process_events())
+			/* User quit */
+			return 0;
 
 		FD_ZERO(&rfds);
 		FD_SET(rdp_socket, &rfds);
@@ -616,7 +705,7 @@
 		}
 
 		if (FD_ISSET(rdp_socket, &rfds))
-			return;
+			return 1;
 	}
 }
 
@@ -633,7 +722,7 @@
 	Pixmap bitmap;
 	uint8 *tdata;
 
-	tdata = translate_image(width, height, data);
+	tdata = (owncolmap ? data : translate_image(width, height, data));
 	bitmap = XCreatePixmap(display, wnd, width, height, depth);
 	image = XCreateImage(display, visual, depth, ZPixmap, 0,
 			     (char *) tdata, width, height, 8, 0);
@@ -641,7 +730,8 @@
 	XPutImage(display, bitmap, gc, image, 0, 0, 0, 0, width, height);
 
 	XFree(image);
-	xfree(tdata);
+	if (!owncolmap)
+		xfree(tdata);
 	return (HBITMAP) bitmap;
 }
 
@@ -651,7 +741,7 @@
 	XImage *image;
 	uint8 *tdata;
 
-	tdata = translate_image(width, height, data);
+	tdata = (owncolmap ? data : translate_image(width, height, data));
 	image = XCreateImage(display, visual, depth, ZPixmap, 0,
 			     (char *) tdata, width, height, 8, 0);
 
@@ -666,7 +756,8 @@
 	}
 
 	XFree(image);
-	xfree(tdata);
+	if (!owncolmap)
+		xfree(tdata);
 }
 
 void
@@ -797,95 +888,125 @@
 		(xc)->blue  = ((c)->blue  << 8) | (c)->blue; \
 		(xc)->flags = DoRed | DoGreen | DoBlue;
 
+
 HCOLOURMAP
 ui_create_colourmap(COLOURMAP * colours)
 {
 	COLOURENTRY *entry;
 	int i, ncolours = colours->ncolours;
-	uint32 *map = xmalloc(sizeof(*colmap) * ncolours);
-	XColor xentry;
-	XColor xc_cache[256];
-	uint32 colour;
-	int colLookup = 256;
-	for (i = 0; i < ncolours; i++)
+	if (!owncolmap)
 	{
-		entry = &colours->colours[i];
-		MAKE_XCOLOR(&xentry, entry);
-
-		if (XAllocColor(display, xcolmap, &xentry) == 0)
+		uint32 *map = xmalloc(sizeof(*colmap) * ncolours);
+		XColor xentry;
+		XColor xc_cache[256];
+		uint32 colour;
+		int colLookup = 256;
+		for (i = 0; i < ncolours; i++)
 		{
-			/* Allocation failed, find closest match. */
-			int j = 256;
-			int nMinDist = 3 * 256 * 256;
-			long nDist = nMinDist;
+			entry = &colours->colours[i];
+			MAKE_XCOLOR(&xentry, entry);
 
-			/* only get the colors once */
-			while (colLookup--)
+			if (XAllocColor(display, xcolmap, &xentry) == 0)
 			{
-				xc_cache[colLookup].pixel = colLookup;
-				xc_cache[colLookup].red = xc_cache[colLookup].green =
-					xc_cache[colLookup].blue = 0;
-				xc_cache[colLookup].flags = 0;
-				XQueryColor(display,
-					    DefaultColormap(display, DefaultScreen(display)),
-					    &xc_cache[colLookup]);
-			}
-			colLookup = 0;
+				/* Allocation failed, find closest match. */
+				int j = 256;
+				int nMinDist = 3 * 256 * 256;
+				long nDist = nMinDist;
 
-			/* approximate the pixel */
-			while (j--)
-			{
-				if (xc_cache[j].flags)
+				/* only get the colors once */
+				while (colLookup--)
 				{
-					nDist = ((long) (xc_cache[j].red >> 8) -
-						 (long) (xentry.red >> 8)) *
-						((long) (xc_cache[j].red >> 8) -
-						 (long) (xentry.red >> 8)) +
-						((long) (xc_cache[j].green >> 8) -
-						 (long) (xentry.green >> 8)) *
-						((long) (xc_cache[j].green >> 8) -
-						 (long) (xentry.green >> 8)) +
-						((long) (xc_cache[j].blue >> 8) -
-						 (long) (xentry.blue >> 8)) *
-						((long) (xc_cache[j].blue >> 8) -
-						 (long) (xentry.blue >> 8));
+					xc_cache[colLookup].pixel = colLookup;
+					xc_cache[colLookup].red = xc_cache[colLookup].green =
+						xc_cache[colLookup].blue = 0;
+					xc_cache[colLookup].flags = 0;
+					XQueryColor(display,
+						    DefaultColormap(display,
+								    DefaultScreen(display)),
+						    &xc_cache[colLookup]);
 				}
-				if (nDist < nMinDist)
+				colLookup = 0;
+
+				/* approximate the pixel */
+				while (j--)
 				{
-					nMinDist = nDist;
-					xentry.pixel = j;
+					if (xc_cache[j].flags)
+					{
+						nDist = ((long) (xc_cache[j].red >> 8) -
+							 (long) (xentry.red >> 8)) *
+							((long) (xc_cache[j].red >> 8) -
+							 (long) (xentry.red >> 8)) +
+							((long) (xc_cache[j].green >> 8) -
+							 (long) (xentry.green >> 8)) *
+							((long) (xc_cache[j].green >> 8) -
+							 (long) (xentry.green >> 8)) +
+							((long) (xc_cache[j].blue >> 8) -
+							 (long) (xentry.blue >> 8)) *
+							((long) (xc_cache[j].blue >> 8) -
+							 (long) (xentry.blue >> 8));
+					}
+					if (nDist < nMinDist)
+					{
+						nMinDist = nDist;
+						xentry.pixel = j;
+					}
 				}
 			}
+			colour = xentry.pixel;
+
+			/* update our cache */
+			if (xentry.pixel < 256)
+			{
+				xc_cache[xentry.pixel].red = xentry.red;
+				xc_cache[xentry.pixel].green = xentry.green;
+				xc_cache[xentry.pixel].blue = xentry.blue;
+
+			}
+
+
+			/* byte swap here to make translate_image faster */
+			map[i] = translate_colour(colour);
 		}
-		colour = xentry.pixel;
+		return map;
+	}
+	else
+	{
+		XColor *xcolours, *xentry;
+		Colormap map;
 
-		/* update our cache */
-		if (xentry.pixel < 256)
+		xcolours = xmalloc(sizeof(XColor) * ncolours);
+		for (i = 0; i < ncolours; i++)
 		{
-			xc_cache[xentry.pixel].red = xentry.red;
-			xc_cache[xentry.pixel].green = xentry.green;
-			xc_cache[xentry.pixel].blue = xentry.blue;
-
+			entry = &colours->colours[i];
+			xentry = &xcolours[i];
+			xentry->pixel = i;
+			MAKE_XCOLOR(xentry, entry);
 		}
 
+		map = XCreateColormap(display, wnd, visual, AllocAll);
+		XStoreColors(display, map, xcolours, ncolours);
 
-		/* byte swap here to make translate_image faster */
-		map[i] = translate_colour(colour);
+		xfree(xcolours);
+		return (HCOLOURMAP) map;
 	}
-
-	return map;
 }
 
 void
 ui_destroy_colourmap(HCOLOURMAP map)
 {
-	xfree(map);
+	if (!owncolmap)
+		xfree(map);
+	else
+		XFreeColormap(display, (Colormap) map);
 }
 
 void
 ui_set_colourmap(HCOLOURMAP map)
 {
-	colmap = map;
+	if (!owncolmap)
+		colmap = map;
+	else
+		XSetWindowColormap(display, wnd, (Colormap) map);
 }
 
 void
@@ -1048,6 +1169,7 @@
 	FILL_RECTANGLE(x, y, cx, cy);
 }
 
+/* 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,
@@ -1062,7 +1184,10 @@
 	XSetStipple(display, gc, (Pixmap) glyph);
 	XSetTSOrigin(display, gc, x, y);
 
-	FILL_RECTANGLE(x, y, cx, cy);
+	if (ownbackstore)
+		XFillRectangle(display, backstore, gc, x, y, cx, cy);
+	else
+		XFillRectangle(display, wnd, gc, x, y, cx, cy);
 
 	XSetFillStyle(display, gc, FillSolid);
 }
@@ -1132,7 +1257,7 @@
 				else
 				{
 					error("this shouldn't be happening\n");
-					break;
+					exit(1);
 				}
 				/* this will move pointer from start to first character after FF command */
 				length -= i + 3;
@@ -1171,8 +1296,15 @@
 				break;
 		}
 	}
-
-
+	if (ownbackstore)
+	{
+		if (boxcx > 1)
+			XCopyArea(display, backstore, wnd, gc, boxx,
+				  boxy, boxcx, boxcy, boxx, boxy);
+		else
+			XCopyArea(display, backstore, wnd, gc, clipx,
+				  clipy, clipcx, clipcy, clipx, clipy);
+	}
 }
 
 void