caesar: WARNS=6, strict bool mode

[bsdgames-darwin.git] / hack / hack.invent.c

diff --git a/hack/hack.invent.c b/hack/hack.invent.c
index 3d7d83771ea7928f6093528dd7eccc2efa03e39f..82284f101765c48b4985e3baa70327798e4c15b4 100644 (file)
--- a/hack/hack.invent.c
+++ b/hack/hack.invent.c
@@ -1,4 +1,4 @@
-/*     $NetBSD: hack.invent.c,v 1.13 2009/06/29 23:05:33 dholland Exp $        */
+/*     $NetBSD: hack.invent.c,v 1.18 2011/08/07 06:03:45 dholland Exp $        */
 
 /*
  * Copyright (c) 1985, Stichting Centrum voor Wiskunde en Informatica,
@@ -63,7 +63,7 @@
 
 #include <sys/cdefs.h>
 #ifndef lint
-__RCSID("$NetBSD: hack.invent.c,v 1.13 2009/06/29 23:05:33 dholland Exp $");
+__RCSID("$NetBSD: hack.invent.c,v 1.18 2011/08/07 06:03:45 dholland Exp $");
 #endif                         /* not lint */
 
 #include <assert.h>
@@ -79,8 +79,9 @@ __RCSID("$NetBSD: hack.invent.c,v 1.13 2009/06/29 23:05:33 dholland Exp $");
 
 static int      lastinvnr = 51;        /* 0 ... 51 */
 
-static void assigninvlet(struct obj *);
 static char *xprname(struct obj *, char);
+static void doinv(const char *);
+static int merged(struct obj *, struct obj *, int);
 
 static void
 assigninvlet(struct obj *otmp)
@@ -236,7 +237,7 @@ freegold(struct gold *gold)
                }
                gtmp->ngold = gold->ngold;
        }
-       free((char *) gold);
+       free(gold);
 }
 
 void
@@ -250,7 +251,7 @@ deltrap(struct trap *trap)
                for (ttmp = ftrap; ttmp->ntrap != trap; ttmp = ttmp->ntrap);
                ttmp->ntrap = trap->ntrap;
        }
-       free((char *) trap);
+       free(trap);
 }
 
 struct wseg    *m_atseg;
@@ -359,7 +360,7 @@ g_at(int x, int y)
 }
 
 /* make dummy object structure containing gold - for temporary use only */
-struct obj     *
+static struct obj *
 mkgoldobj(long q)
 {
        struct obj     *otmp;
@@ -498,7 +499,7 @@ getobj(const char *let, const char *word)
                                continue;
                        /* he typed a letter (not a space) to more() */
                } else if (ilet == '*') {
-                       doinv((char *) 0);
+                       doinv(NULL);
                        if (!(ilet = morc))
                                continue;
                        /* ... */
@@ -542,7 +543,7 @@ getobj(const char *let, const char *word)
        return (otmp);
 }
 
-int
+static int
 ckunpaid(struct obj *otmp)
 {
        return (otmp->unpaid);
@@ -654,7 +655,7 @@ askchain(struct obj *objchn, char *olets, int allflag,
                if (ckfn && !(*ckfn) (otmp))
                        continue;
                if (!allflag) {
-                       pline(xprname(otmp, ilet));
+                       pline("%s", xprname(otmp, ilet));
                        addtopl(" [nyaq]? ");
                        sym = readchar();
                } else
@@ -663,10 +664,12 @@ askchain(struct obj *objchn, char *olets, int allflag,
                switch (sym) {
                case 'a':
                        allflag = 1;
+                       /* FALLTHROUGH */
                case 'y':
                        cnt += (*fn) (otmp);
                        if (--max == 0)
                                goto ret;
+                       break;
                case 'n':
                default:
                        break;
@@ -680,7 +683,7 @@ ret:
 }
 
 /* should of course only be called for things in invent */
-char
+static char
 obj_to_let(struct obj *obj)
 {
        struct obj     *otmp;
@@ -698,10 +701,10 @@ obj_to_let(struct obj *obj)
 void
 prinv(struct obj *obj)
 {
-       pline(xprname(obj, obj_to_let(obj)));
+       pline("%s", xprname(obj, obj_to_let(obj)));
 }
 
-static char    *
+static char *
 xprname(struct obj *obj, char let)
 {
        static char     li[BUFSZ];
@@ -715,14 +718,14 @@ xprname(struct obj *obj, char let)
 int
 ddoinv(void)
 {
-       doinv((char *) 0);
+       doinv(NULL);
        return (0);
 }
 
 /* called with 0 or "": all objects in inventory */
 /* otherwise: all objects with (serial) letter in lets */
-void
-doinv(char *lets)
+static void
+doinv(const char *lets)
 {
        struct obj     *otmp;
        char            ilet;
@@ -735,7 +738,7 @@ doinv(char *lets)
                pline("Not carrying anything.");
                return;
        }
-       cornline(0, (char *) 0);
+       cornline(0, NULL);
        ilet = 'a';
        for (otmp = invent; otmp; otmp = otmp->nobj) {
                if (flags.invlet_constant)
@@ -882,10 +885,10 @@ dolook(void)
        }
        if (ct == 1 && !gold) {
                pline("You %s here %s.", verb, doname(otmp0));
-               cornline(3, (char *) 0);
+               cornline(3, NULL);
        }
        if (ct > 1)
-               cornline(2, (char *) 0);
+               cornline(2, NULL);
        return (!!Blind);
 }
 
@@ -901,7 +904,7 @@ stackobj(struct obj *obj)
 }
 
 /* merge obj with otmp and delete obj if types agree */
-int
+static int
 merged(struct obj *otmp, struct obj *obj, int lose)
 {
        if (obj->otyp == otmp->otyp &&
@@ -928,7 +931,7 @@ static long goldcounted;
  * it may take a while before you have counted it all.
  * [Bug: d$ and pickup still tell you how much it was.]
  */
-int
+static int
 countgold(void)
 {
        if ((goldcounted += 100 * (u.ulevel + 1)) >= u.ugold) {