Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/games/cgram cgram: adjust style to survive lint's strict boo...



details:   https://anonhg.NetBSD.org/src/rev/dddf5f7de518
branches:  trunk
changeset: 1018936:dddf5f7de518
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Feb 21 16:05:44 2021 +0000

description:
cgram: adjust style to survive lint's strict bool mode

The biggest change is the return type of substitute().  Before, 0 meant
success, now false means failure.

No functional change.

diffstat:

 games/cgram/Makefile |  15 ++++++++-------
 games/cgram/cgram.c  |  33 +++++++++++++++++----------------
 2 files changed, 25 insertions(+), 23 deletions(-)

diffs (148 lines):

diff -r ee50f5c949bf -r dddf5f7de518 games/cgram/Makefile
--- a/games/cgram/Makefile      Sun Feb 21 15:45:30 2021 +0000
+++ b/games/cgram/Makefile      Sun Feb 21 16:05:44 2021 +0000
@@ -1,10 +1,11 @@
-# $NetBSD: Makefile,v 1.1 2013/08/04 05:42:47 dholland Exp $
+# $NetBSD: Makefile,v 1.2 2021/02/21 16:05:44 rillig Exp $
 
-PROG=cgram
-DPADD=${LIBCURSES} ${LIBTERMINFO}
-LDADD=-lcurses -lterminfo
-SRCS=cgram.c
-MAN=cgram.6
-HIDEGAME=hidegame
+PROG=          cgram
+DPADD=         ${LIBCURSES} ${LIBTERMINFO}
+LDADD=         -lcurses -lterminfo
+SRCS=          cgram.c
+MAN=           cgram.6
+HIDEGAME=      hidegame
+LINTFLAGS+=    -T
 
 .include <bsd.prog.mk>
diff -r ee50f5c949bf -r dddf5f7de518 games/cgram/cgram.c
--- a/games/cgram/cgram.c       Sun Feb 21 15:45:30 2021 +0000
+++ b/games/cgram/cgram.c       Sun Feb 21 16:05:44 2021 +0000
@@ -33,6 +33,7 @@
 #include <ctype.h>
 #include <curses.h>
 #include <err.h>
+#include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -99,22 +100,22 @@
 readquote(void)
 {
        FILE *f = popen(_PATH_FORTUNE, "r");
-       if (!f) {
+       if (f == NULL) {
                err(1, "%s", _PATH_FORTUNE);
        }
 
        char buf[128], buf2[8 * sizeof(buf)];
-       while (fgets(buf, sizeof(buf), f)) {
+       while (fgets(buf, sizeof buf, f) != NULL) {
                char *s = strrchr(buf, '\n');
-               assert(s);
+               assert(s != NULL);
                assert(strlen(s) == 1);
-               *s = 0;
+               *s = '\0';
 
                int i, j;
-               for (i = j = 0; buf[i]; i++) {
+               for (i = j = 0; buf[i] != '\0'; i++) {
                        if (buf[i] == '\t') {
                                buf2[j++] = ' ';
-                               while (j % 8)
+                               while (j % 8 != 0)
                                        buf2[j++] = ' ';
                        } else if (buf[i] == '\b') {
                                if (j > 0)
@@ -123,7 +124,7 @@
                                buf2[j++] = buf[i];
                        }
                }
-               buf2[j] = 0;
+               buf2[j] = '\0';
 
                stringarray_add(&lines, buf2);
                stringarray_add(&sollines, buf2);
@@ -146,7 +147,7 @@
        }
 
        for (int y = 0; y < lines.num; y++) {
-               for (unsigned x = 0; lines.v[y][x]; x++) {
+               for (unsigned x = 0; lines.v[y][x] != '\0'; x++) {
                        if (islower((unsigned char)lines.v[y][x])) {
                                int q = lines.v[y][x] - 'a';
                                lines.v[y][x] = 'a' + key[q];
@@ -159,19 +160,19 @@
        }
 }
 
-static int
+static bool
 substitute(int ch)
 {
        assert(cury >= 0 && cury < lines.num);
        if (curx >= strlen(lines.v[cury])) {
                beep();
-               return -1;
+               return false;
        }
 
        int och = lines.v[cury][curx];
        if (!isalpha((unsigned char)och)) {
                beep();
-               return -1;
+               return false;
        }
 
        int loch = tolower((unsigned char)och);
@@ -180,7 +181,7 @@
        int uch = toupper((unsigned char)ch);
 
        for (int y = 0; y < lines.num; y++) {
-               for (unsigned x = 0; lines.v[y][x]; x++) {
+               for (unsigned x = 0; lines.v[y][x] != '\0'; x++) {
                        if (lines.v[y][x] == loch) {
                                lines.v[y][x] = lch;
                        } else if (lines.v[y][x] == uoch) {
@@ -192,7 +193,7 @@
                        }
                }
        }
-       return 0;
+       return true;
 }
 
 ////////////////////////////////////////////////////////////
@@ -206,7 +207,7 @@
                move(i, 0);
                int ln = i + scrolldown;
                if (ln < lines.num) {
-                       for (unsigned j = 0; lines.v[i][j]; j++) {
+                       for (unsigned j = 0; lines.v[i][j] != '\0'; j++) {
                                int ch = lines.v[i][j];
                                if (ch != sollines.v[i][j] &&
                                    isalpha((unsigned char)ch)) {
@@ -325,7 +326,7 @@
                        break;
                default:
                        if (isalpha(ch)) {
-                               if (!substitute(ch)) {
+                               if (substitute(ch)) {
                                        if (curx < strlen(lines.v[cury])) {
                                                curx++;
                                        }
@@ -364,7 +365,7 @@
        encode();
        opencurses();
 
-       keypad(stdscr, TRUE);
+       keypad(stdscr, true);
        loop();
 
        closecurses();



Home | Main Index | Thread Index | Old Index