[PATCH:libXcursor] Mark bitmasks as unsigned ints
Alan Coopersmith
alan.coopersmith at oracle.com
Sun May 22 12:57:28 PDT 2011
Clears Sun compiler warnings from shifting 8 bits by 24 bits:
"cursor.c", line 215: warning: integer overflow detected: op "<<"
"cursor.c", line 280: warning: integer overflow detected: op "<<"
Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
src/cursor.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/cursor.c b/src/cursor.c
index 3de3c8e..7da62ef 100644
--- a/src/cursor.c
+++ b/src/cursor.c
@@ -212,7 +212,7 @@ _XcursorAverageColor (XcursorPixel *pixels, int npixels)
green += (p >> 8) & 0xff;
blue += (p >> 0) & 0xff;
}
- return (0xff << 24) | ((red/npixels) << 16) | ((green/npixels) << 8) | (blue/npixels);
+ return (0xffU << 24) | ((red/npixels) << 16) | ((green/npixels) << 8) | (blue/npixels);
}
typedef struct XcursorCoreCursor {
@@ -277,7 +277,7 @@ _XcursorHeckbertMedianCut (const XcursorImage *image, XcursorCoreCursor *core)
if (green > max_green) max_green = green;
if (blue < min_blue) min_blue = blue;
if (blue > max_blue) max_blue = blue;
- p = ((0xff << 24) | (red << 16) |
+ p = ((0xffU << 24) | (red << 16) |
(green << 8) | (blue << 0));
*pc++ = p;
}
--
1.7.3.2
More information about the xorg-devel
mailing list