From 490defa042876b729180786e9af14743a1fa7dd5 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <icculus@icculus.org>
Date: Mon, 3 Oct 2005 08:12:32 +0000
Subject: [PATCH] Fixed compiler warnings in Watcom C.

---
 test/testcdrom.c     |  3 ++-
 test/testkeys.c      |  2 ++
 test/testtypes.c     | 20 ++++++++++++++++----
 test/testver.c       |  1 +
 test/torturethread.c |  2 +-
 5 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/test/testcdrom.c b/test/testcdrom.c
index 8f9a50b1b..9a962a82b 100644
--- a/test/testcdrom.c
+++ b/test/testcdrom.c
@@ -2,8 +2,9 @@
 /* Test the SDL CD-ROM audio functions */
 
 #include <stdlib.h>
-#include <stdio.h>
 #include <ctype.h>
+#include <stdlib.h>
+#include <string.h>
 
 #include "SDL.h"
 
diff --git a/test/testkeys.c b/test/testkeys.c
index 42f931daa..e320c6dff 100644
--- a/test/testkeys.c
+++ b/test/testkeys.c
@@ -3,6 +3,8 @@
 
 #include <stdio.h>
 #include <ctype.h>
+#include <stdlib.h>
+#include <string.h>
 
 #include "SDL.h"
 
diff --git a/test/testtypes.c b/test/testtypes.c
index 3ed73f077..485360791 100644
--- a/test/testtypes.c
+++ b/test/testtypes.c
@@ -1,8 +1,20 @@
 
 #include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 #include "SDL_main.h"
 #include "SDL_types.h"
 
+/*
+ * Watcom C flags these as Warning 201: "Unreachable code" if you just
+ *  compare them directly, so we push it through a function to keep the
+ *  compiler quiet.  --ryan.
+ */
+static int badsize(size_t sizeoftype, size_t hardcodetype)
+{
+    return sizeoftype != hardcodetype;
+}
+
 int main(int argc, char *argv[])
 {
 	int error = 0;
@@ -11,26 +23,26 @@ int main(int argc, char *argv[])
 	if ( argv[1] && (strcmp(argv[1], "-q") == 0) )
 		verbose = 0;
 
-	if ( sizeof(Uint8) != 1 ) {
+	if ( badsize(sizeof(Uint8), 1) ) {
 		if ( verbose )
 			printf("sizeof(Uint8) != 1, instead = %d\n",
 								sizeof(Uint8));
 		++error;
 	}
-	if ( sizeof(Uint16) != 2 ) {
+	if ( badsize(sizeof(Uint16), 2) ) {
 		if ( verbose )
 			printf("sizeof(Uint16) != 2, instead = %d\n",
 								sizeof(Uint16));
 		++error;
 	}
-	if ( sizeof(Uint32) != 4 ) {
+	if ( badsize(sizeof(Uint32), 4) ) {
 		if ( verbose )
 			printf("sizeof(Uint32) != 4, instead = %d\n",
 								sizeof(Uint32));
 		++error;
 	}
 #ifdef SDL_HAS_64BIT_TYPE
-	if ( sizeof(Uint64) != 8 ) {
+	if ( badsize(sizeof(Uint64), 8) ) {
 		if ( verbose )
 			printf("sizeof(Uint64) != 8, instead = %d\n",
 								sizeof(Uint64));
diff --git a/test/testver.c b/test/testver.c
index 4155a0941..4f45ffe57 100644
--- a/test/testver.c
+++ b/test/testver.c
@@ -4,6 +4,7 @@
 */
 
 #include <stdio.h>
+#include <stdlib.h>
 
 #include "SDL.h"
 
diff --git a/test/torturethread.c b/test/torturethread.c
index 444e8d1e5..b12d9cf51 100644
--- a/test/torturethread.c
+++ b/test/torturethread.c
@@ -22,7 +22,7 @@ static void quit(int rc)
 
 int SubThreadFunc(void *data) {
 	while(! *(int volatile *)data) {
-		; /*SDL_Delay(10); /* do nothing */
+		; /*SDL_Delay(10);*/  /* do nothing */
 	}
 	return 0;
 }