Module Name:    src
Committed By:   dholland
Date:           Sun Jul 26 03:02:38 UTC 2009

Modified Files:
        src/games/bcd: Makefile bcd.c

Log Message:
Remove the need for -Wno-pointer-sign. Object diffs checked.


To generate a diff of this commit:
cvs rdiff -u -r1.7 -r1.8 src/games/bcd/Makefile
cvs rdiff -u -r1.15 -r1.16 src/games/bcd/bcd.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/games/bcd/Makefile
diff -u src/games/bcd/Makefile:1.7 src/games/bcd/Makefile:1.8
--- src/games/bcd/Makefile:1.7	Fri Aug 29 00:02:22 2008
+++ src/games/bcd/Makefile	Sun Jul 26 03:02:38 2009
@@ -1,4 +1,4 @@
-#	$NetBSD: Makefile,v 1.7 2008/08/29 00:02:22 gmcgarry Exp $
+#	$NetBSD: Makefile,v 1.8 2009/07/26 03:02:38 dholland Exp $
 #	@(#)Makefile	8.1 (Berkeley) 5/31/93
 
 PROG=	bcd
@@ -7,7 +7,3 @@
 HIDEGAME=hidegame
 
 .include <bsd.prog.mk>
-
-.if (defined(HAVE_GCC) && ${HAVE_GCC} == 4) || defined(HAVE_PCC)
-COPTS.bcd.c+=	-Wno-pointer-sign
-.endif

Index: src/games/bcd/bcd.c
diff -u src/games/bcd/bcd.c:1.15 src/games/bcd/bcd.c:1.16
--- src/games/bcd/bcd.c:1.15	Sun Jul 20 01:03:21 2008
+++ src/games/bcd/bcd.c	Sun Jul 26 03:02:38 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: bcd.c,v 1.15 2008/07/20 01:03:21 lukem Exp $	*/
+/*	$NetBSD: bcd.c,v 1.16 2009/07/26 03:02:38 dholland Exp $	*/
 
 /*
  * Copyright (c) 1989, 1993
@@ -42,7 +42,7 @@
 #if 0
 static char sccsid[] = "@(#)bcd.c	8.2 (Berkeley) 3/20/94";
 #else
-__RCSID("$NetBSD: bcd.c,v 1.15 2008/07/20 01:03:21 lukem Exp $");
+__RCSID("$NetBSD: bcd.c,v 1.16 2009/07/26 03:02:38 dholland Exp $");
 #endif
 #endif /* not lint */
 
@@ -155,7 +155,7 @@
 {
 	static const char rowchars[] = "   123456789";
 	int i, row;
-	unsigned char *p;
+	char *p;
 
 	/* ruthlessly remove newlines and truncate at 48 characters. */
 	if ((p = strchr(str, '\n')))
@@ -166,8 +166,8 @@
 
 	/* make string upper case. */
 	for (p = str; *p; ++p)
-		if (isascii(*p) && islower(*p))
-			*p = toupper(*p);
+		if (isascii((unsigned char)*p) && islower((unsigned char)*p))
+			*p = toupper((unsigned char) *p);
 
 	 /* top of card */
 	putchar(' ');
@@ -182,7 +182,7 @@
 	p = str;
 	putchar('/');
 	for (i = 1; *p; i++, p++)
-		if (holes[(int)*p])
+		if (holes[(unsigned char)*p])
 			putchar(*p);
 		else
 			putchar(' ');
@@ -200,7 +200,7 @@
 	for (row = 0; row <= 11; ++row) {
 		putchar('|');
 		for (i = 0, p = str; *p; i++, p++) {
-			if (bit(holes[(int)*p], 11 - row))
+			if (bit(holes[(unsigned char)*p], 11 - row))
 				putchar(']');
 			else
 				putchar(rowchars[row]);

Reply via email to