[PATCH:xscope 10/24] Convert ILong, IShort, etal to return C99 uint*_t types

Alan Coopersmith alan.coopersmith at oracle.com
Fri Aug 31 22:17:52 PDT 2012


Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
 server.c |    8 ++++----
 x11.h    |   10 ++++++----
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/server.c b/server.c
index 3369460..5226a03 100644
--- a/server.c
+++ b/server.c
@@ -138,7 +138,7 @@ pad(long n)
     return ((n + 3) & ~0x3);
 }
 
-unsigned long
+uint32_t
 ILong(const unsigned char buf[])
 {
     /* check for byte-swapping */
@@ -147,7 +147,7 @@ ILong(const unsigned char buf[])
     return ((((((buf[0] << 8) | buf[1]) << 8) | buf[2]) << 8) | buf[3]);
 }
 
-unsigned short
+uint16_t
 IShort(const unsigned char buf[])
 {
     /* check for byte-swapping */
@@ -156,14 +156,14 @@ IShort(const unsigned char buf[])
     return ((buf[0] << 8) | buf[1]);
 }
 
-unsigned short
+uint16_t
 IChar2B(const unsigned char buf[])
 {
     /* CHAR2B is like an IShort, but not byte-swapped */
     return ((buf[0] << 8) | buf[1]);
 }
 
-unsigned short
+uint8_t
 IByte(const unsigned char buf[])
 {
     return (buf[0]);
diff --git a/x11.h b/x11.h
index 4fba039..2a8b6bb 100644
--- a/x11.h
+++ b/x11.h
@@ -51,6 +51,8 @@
 #ifndef XSCOPE_X11_H
 #define XSCOPE_X11_H
 
+#include <inttypes.h>
+
 /* Some field contents are constants, not just types */
 
 #define CONST1(n)  CARD8
@@ -531,10 +533,10 @@ extern void PrintValueRec(unsigned long key, unsigned long cmask, short ctype);
 
 /* declaration of the types of some common functions */
 
-extern unsigned long ILong(const unsigned char *buf);
-extern unsigned short IShort(const unsigned char *buf);
-extern unsigned short IChar2B(const unsigned char *buf);
-extern unsigned short IByte(const unsigned char *buf);
+extern uint32_t ILong(const unsigned char *buf);
+extern uint16_t IShort(const unsigned char *buf);
+extern uint16_t IChar2B(const unsigned char *buf);
+extern uint8_t IByte(const unsigned char *buf);
 extern Boolean IBool(const unsigned char *buf);
 
 extern int PrintString8(const unsigned char *buf, int number, const char *name);
-- 
1.7.9.2



More information about the xorg-devel mailing list