[src/trunk]: src/games/larn eliminate/document non-literal format strings

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/2506320ad85b
branches:  trunk
changeset: 768415:2506320ad85b
user:      christos <christos%NetBSD.org@localhost>
date:      Tue Aug 16 11:19:41 2011 +0000

description:
eliminate/document non-literal format strings

diffstat:

 games/larn/Makefile |   5 ++++-
 games/larn/movem.c  |  22 ++++++++++------------
 2 files changed, 14 insertions(+), 13 deletions(-)

diffs (76 lines):

diff -r 421dffc528f3 -r 2506320ad85b games/larn/Makefile
--- a/games/larn/Makefile       Tue Aug 16 11:14:04 2011 +0000
+++ b/games/larn/Makefile       Tue Aug 16 11:19:41 2011 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.20 2010/02/03 15:34:39 roy Exp $
+#      $NetBSD: Makefile,v 1.21 2011/08/16 11:19:41 christos Exp $
 #      @(#)Makefile    5.12 (Berkeley) 5/30/93
 
 # EXTRA
@@ -74,4 +74,7 @@
 FILESDIR=/usr/share/games/larn
 .endif
 
+COPTS.display.c += -Wno-format-nonliteral
+COPTS.monster.c += -Wno-format-nonliteral
+
 .include <bsd.prog.mk>
diff -r 421dffc528f3 -r 2506320ad85b games/larn/movem.c
--- a/games/larn/movem.c        Tue Aug 16 11:14:04 2011 +0000
+++ b/games/larn/movem.c        Tue Aug 16 11:19:41 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: movem.c,v 1.7 2009/08/12 08:04:05 dholland Exp $       */
+/*     $NetBSD: movem.c,v 1.8 2011/08/16 11:19:41 christos Exp $       */
 
 /*
  * movem.c (move monster)              Larn is copyrighted 1986 by Noah Morgan.
@@ -12,7 +12,7 @@
  */
 #include <sys/cdefs.h>
 #ifndef lint
-__RCSID("$NetBSD: movem.c,v 1.7 2009/08/12 08:04:05 dholland Exp $");
+__RCSID("$NetBSD: movem.c,v 1.8 2011/08/16 11:19:41 christos Exp $");
 #endif                         /* not lint */
 
 #include "header.h"
@@ -278,7 +278,7 @@
        int             aa, bb, cc, dd;
 {
        int    tmp, i, flag;
-       const char *who = NULL, *p;
+       const char *who = NULL;
 
        flag = 0;               /* set to 1 if monster hit by arrow trap */
        if ((cc == playerx) && (dd == playery)) {
@@ -364,24 +364,22 @@
        if (c[BLINDCOUNT])
                return;         /* if blind don't show where monsters are        */
        if (know[cc][dd] & 1) {
-               p = 0;
                if (flag)
                        cursors();
                switch (flag) {
                case 1:
-                       p = "\n%s hits the %s";
+                       lprintf("\n%s hits the %s", who, monster[tmp].name);
+                       beep();
                        break;
                case 2:
-                       p = "\n%s hits and kills the %s";
+                       lprintf("\n%s hits and kills the %s",
+                           who, monster[tmp].name);
+                       beep();
                        break;
                case 3:
-                       p = "\nThe %s%s gets teleported";
-                       who = "";
+                       lprintf("\nThe %s gets teleported", monster[tmp].name);
+                       beep();
                        break;
-               };
-               if (p) {
-                       lprintf(p, who, monster[tmp].name);
-                       beep();
                }
        }
        /*