From 99e7f9d8ca8b6f09e88efc4fc394d0fb3fa1754f Mon Sep 17 00:00:00 2001
From: jsorg71 <jsorg71>
Date: Sat, 28 Apr 2007 18:23:08 +0000
Subject: [PATCH] need 24 bit RGB and BGR macros

---
 common/defines.h   |  3 ++-
 xrdp/xrdp_bitmap.c |  6 +++---
 xrdp/xrdp_wm.c     | 16 ++++++++--------
 3 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/common/defines.h b/common/defines.h
index 5ce89f19..f5e2e7d2 100644
--- a/common/defines.h
+++ b/common/defines.h
@@ -64,7 +64,8 @@
 )
 #define COLOR15(r, g, b) ((((r) >> 3) << 10) | (((g) >> 3) << 5) | ((b) >> 3))
 #define COLOR16(r, g, b) ((((r) >> 3) << 11) | (((g) >> 2) << 5) | ((b) >> 3))
-#define COLOR24(r, g, b) ((r) | ((g) << 8) | ((b) << 16))
+#define COLOR24RGB(r, g, b) (((r) << 16) | ((g) << 8) | (b))
+#define COLOR24BGR(r, g, b) (((b) << 16) | ((g) << 8) | (r))
 #define SPLITCOLOR15(r, g, b, c) \
 { \
   r = (((c) >> 7) & 0xf8) | (((c) >> 12) & 0x7); \
diff --git a/xrdp/xrdp_bitmap.c b/xrdp/xrdp_bitmap.c
index d2557c81..122fb268 100644
--- a/xrdp/xrdp_bitmap.c
+++ b/xrdp/xrdp_bitmap.c
@@ -429,9 +429,9 @@ xrdp_bitmap_load(struct xrdp_bitmap* self, const char* filename, int* palette)
           }
           else if (self->bpp == 24)
           {
-            color = COLOR24((color & 0xff0000) >> 16,
-                            (color & 0x00ff00) >> 8,
-                            (color & 0x0000ff) >> 0);
+            //color = COLOR24((color & 0xff0000) >> 16,
+            //                (color & 0x00ff00) >> 8,
+            //                (color & 0x0000ff) >> 0);
           }
           xrdp_bitmap_set_pixel(self, j, i, color);
         }
diff --git a/xrdp/xrdp_wm.c b/xrdp/xrdp_wm.c
index bff8b004..2c879e0e 100644
--- a/xrdp/xrdp_wm.c
+++ b/xrdp/xrdp_wm.c
@@ -309,14 +309,14 @@ xrdp_wm_load_static_colors(struct xrdp_wm* self)
   }
   else if (self->screen->bpp == 24)
   {
-    self->black     = COLOR24(0, 0, 0);
-    self->grey      = COLOR24(0xc0, 0xc0, 0xc0);
-    self->dark_grey = COLOR24(0x80, 0x80, 0x80);
-    self->blue      = COLOR24(0x00, 0x00, 0xff);
-    self->dark_blue = COLOR24(0x00, 0x00, 0x7f);
-    self->white     = COLOR24(0xff, 0xff, 0xff);
-    self->red       = COLOR24(0xff, 0x00, 0x00);
-    self->green     = COLOR24(0x00, 0xff, 0x00);
+    self->black     = COLOR24BGR(0, 0, 0);
+    self->grey      = COLOR24BGR(0xc0, 0xc0, 0xc0);
+    self->dark_grey = COLOR24BGR(0x80, 0x80, 0x80);
+    self->blue      = COLOR24BGR(0x00, 0x00, 0xff);
+    self->dark_blue = COLOR24BGR(0x00, 0x00, 0x7f);
+    self->white     = COLOR24BGR(0xff, 0xff, 0xff);
+    self->red       = COLOR24BGR(0xff, 0x00, 0x00);
+    self->green     = COLOR24BGR(0x00, 0xff, 0x00);
   }
   return 0;
 }