--- sourceforge.net/trunk/rdesktop/xwin.c	2003/10/11 19:12:10	483
+++ sourceforge.net/trunk/rdesktop/xwin.c	2003/10/28 09:02:46	521
@@ -49,6 +49,7 @@
 static XIC g_IC;
 static XModifierKeymap *g_mod_map;
 static Cursor g_current_cursor;
+static HCURSOR g_null_cursor;
 static Atom g_protocol_atom, g_kill_atom;
 static BOOL g_focused;
 static BOOL g_mouse_in_wnd;
@@ -69,6 +70,7 @@
 #ifdef WITH_RDPSND
 extern int g_dsp_fd;
 extern BOOL g_dsp_busy;
+extern BOOL g_rdpsnd;
 #endif
 
 /* MWM decorations */
@@ -273,6 +275,7 @@
 					colour = make_colour16(split_colour24(colour));
 					break;
 				case 24:
+					colour = make_colour24(split_colour24(colour));
 					break;
 				case 32:
 					colour = make_colour32(split_colour24(colour));
@@ -291,10 +294,25 @@
 }
 
 static void
-translate8to16(uint8 * data, uint16 * out, uint16 * end)
+translate8to16(uint8 * data, uint8 * out, uint8 * end)
 {
+	uint16 value;
+
 	while (out < end)
-		*(out++) = (uint16) g_colmap[*(data++)];
+	{
+		value = (uint16) g_colmap[*(data++)];
+		
+		if (g_xserver_be)
+		{
+			*(out++) = value >> 8;
+			*(out++) = value;
+		}
+		else
+		{
+			*(out++) = value;
+			*(out++) = value >> 8;
+		}
+	}
 }
 
 /* little endian - conversion happens when colourmap is built */
@@ -306,17 +324,46 @@
 	while (out < end)
 	{
 		value = g_colmap[*(data++)];
-		*(out++) = value;
-		*(out++) = value >> 8;
-		*(out++) = value >> 16;
+		
+		if (g_xserver_be)
+		{
+			*(out++) = value >> 16;
+			*(out++) = value >> 8;
+			*(out++) = value;
+		}
+		else
+		{
+			*(out++) = value;
+			*(out++) = value >> 8;
+			*(out++) = value >> 16;
+		}
 	}
 }
 
 static void
-translate8to32(uint8 * data, uint32 * out, uint32 * end)
+translate8to32(uint8 * data, uint8 * out, uint8 * end)
 {
+	uint32 value;
+
 	while (out < end)
-		*(out++) = g_colmap[*(data++)];
+	{
+		value = g_colmap[*(data++)];
+
+		if (g_xserver_be)
+		{
+			*(out++) = value >> 24;
+			*(out++) = value >> 16;
+			*(out++) = value >> 8;
+			*(out++) = value;
+		}
+		else
+		{
+			*(out++) = value;
+			*(out++) = value >> 8;
+			*(out++) = value >> 16;
+			*(out++) = value >> 24;
+		}
+	}
 }
 
 /* todo the remaining translate function might need some big endian check ?? */
@@ -333,7 +380,7 @@
 
 		if (g_host_be)
 		{
-			BSWAP16(pixel)
+			BSWAP16(pixel);
 		}
 
 		value = make_colour16(split_colour15(pixel));
@@ -363,7 +410,7 @@
 
 		if (g_host_be)
 		{
-			BSWAP16(pixel)
+			BSWAP16(pixel);
 		}
 
 		value = make_colour24(split_colour15(pixel));
@@ -457,7 +504,7 @@
 
 		if (g_host_be)
 		{
-			BSWAP16(pixel)
+			BSWAP16(pixel);
 		}
 
 		value = make_colour24(split_colour16(pixel));
@@ -489,8 +536,7 @@
 
 		if (g_host_be)
 		{
-			BSWAP16(pixel)
-		}
+		BSWAP16(pixel)}
 
 		value = make_colour32(split_colour16(pixel));
 
@@ -500,15 +546,15 @@
 			*(out++) = value >> 16;
 			*(out++) = value >> 8;
 			*(out++) = value;
- 		}
- 		else
- 		{
+		}
+		else
+		{
 			*(out++) = value;
 			*(out++) = value >> 8;
 			*(out++) = value >> 16;
 			*(out++) = value >> 24;
- 		}
- 	}
+		}
+	}
 }
 
 static void
@@ -627,13 +673,13 @@
 					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;
@@ -707,7 +753,11 @@
 		return False;
 	}
 
-	if (g_owncolmap != True)
+	/* private colour map code only works for 8 bpp */
+	if (g_owncolmap && (g_bpp > 8))
+		g_owncolmap = False;
+
+	if (!g_owncolmap)
 	{
 		g_xcolmap = DefaultColormapOfScreen(g_screen);
 		if (g_depth <= 8)
@@ -723,7 +773,16 @@
 	g_host_be = !(BOOL) (*(uint8 *) (&test));
 	g_xserver_be = (ImageByteOrder(g_display) == MSBFirst);
 
-	if ((g_width == 0) || (g_height == 0))
+	/*
+	 * Determine desktop size
+	 */
+	if (g_width < 0)
+	{
+		/* Percent of screen */
+		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;
@@ -740,8 +799,7 @@
 			g_height = 600;
 		}
 	}
-
-	if (g_fullscreen)
+	else if (g_fullscreen)
 	{
 		g_width = WidthOfScreen(g_screen);
 		g_height = HeightOfScreen(g_screen);
@@ -763,14 +821,14 @@
 
 	g_mod_map = XGetModifierMapping(g_display);
 
+	xkeymap_init();
+
 	if (g_enable_compose)
 		g_IM = XOpenIM(g_display, NULL, NULL, NULL);
 
-	xkeymap_init();
 	xclip_init();
 
-	/* todo take this out when high colour is done */
-	printf("server bpp %d client bpp %d depth %d\n", g_server_bpp, g_bpp, g_depth);
+	DEBUG_RDP5(("server bpp %d client bpp %d depth %d\n", g_server_bpp, g_bpp, g_depth));
 
 	return True;
 }
@@ -791,6 +849,9 @@
 	g_display = NULL;
 }
 
+#define NULL_POINTER_MASK	"\x80"
+#define NULL_POINTER_DATA	"\x0\x0\x0"
+	
 BOOL
 ui_create_window(void)
 {
@@ -875,12 +936,17 @@
 	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 */
+	g_null_cursor = ui_create_cursor(0, 0, 1, 1, NULL_POINTER_MASK, NULL_POINTER_DATA);
+
 	return True;
 }
 
 void
 ui_destroy_window(void)
 {
+	ui_destroy_cursor(g_null_cursor);
+	
 	if (g_IC != NULL)
 		XDestroyIC(g_IC);
 
@@ -1202,7 +1268,7 @@
 		{
 			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))
@@ -1236,11 +1302,24 @@
 	XImage *image;
 	Pixmap bitmap;
 	uint8 *tdata;
+	int bitmap_pad;
+
+	if (g_server_bpp == 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);
 
@@ -1255,9 +1334,23 @@
 {
 	XImage *image;
 	uint8 *tdata;
+	int bitmap_pad;
+
+	if (g_server_bpp == 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)
 	{
@@ -1396,6 +1489,12 @@
 	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; \
@@ -1596,8 +1695,8 @@
 		case 2:	/* Hatch */
 			fill = (Pixmap) ui_create_glyph(8, 8,
 							hatch_patterns + brush->pattern[0] * 8);
-			SET_FOREGROUND(bgcolour);
-			SET_BACKGROUND(fgcolour);
+			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);