mtree(8): Raise WARNS to 6 and silence all warnings.
authorSascha Wildner <saw@online.de>
Sun, 22 Mar 2009 22:23:38 +0000 (23:23 +0100)
committerSascha Wildner <saw@online.de>
Sun, 22 Mar 2009 22:23:38 +0000 (23:23 +0100)
usr.sbin/mtree/Makefile
usr.sbin/mtree/compare.c
usr.sbin/mtree/create.c
usr.sbin/mtree/excludes.c
usr.sbin/mtree/extern.h
usr.sbin/mtree/misc.c
usr.sbin/mtree/mtree.c
usr.sbin/mtree/verify.c

index c563c93..819fe39 100644 (file)
@@ -7,6 +7,7 @@
 PROG=  mtree
 MAN=   mtree.8
 SRCS=  compare.c crc.c create.c excludes.c misc.c mtree.c spec.c verify.c
+WARNS?=        6
 
 .if defined(BOOTSTRAPPING)
 .PATH: ${.CURDIR}/../../lib/libc/gen
index 5fddafd..89659a5 100644 (file)
 #include "mtree.h"
 #include "extern.h"
 
-extern int uflag;
-extern int lineno;
-
-static char *ftype(u_int);
+static const char *ftype(u_int);
 
 #define        INDENTNAMELEN   8
 #define        LABEL \
@@ -69,13 +66,12 @@ static char *ftype(u_int);
        }
 
 int
-compare(char *name, NODE *s, FTSENT *p)
+compare(NODE *s, FTSENT *p)
 {
-       extern int uflag;
        u_long len, val;
        int fd, label;
-       char *cp, *tab = "";
-       char *fflags;
+       char *cp, *fflags;
+       const char *tab = "";
 
        label = 0;
        switch(s->type) {
@@ -167,8 +163,8 @@ typeerr:            LABEL;
        if (s->flags & F_SIZE && s->st_size != p->fts_statp->st_size &&
                !S_ISDIR(p->fts_statp->st_mode)) {
                LABEL;
-               printf("%ssize expected %qd found %qd\n",
-                   tab, s->st_size, p->fts_statp->st_size);
+               printf("%ssize expected %jd found %jd\n", tab,
+                   (intmax_t)s->st_size, (intmax_t)p->fts_statp->st_size);
                tab = "\t";
        }
        /*
@@ -297,7 +293,7 @@ typeerr:            LABEL;
        return (label);
 }
 
-char *
+const char *
 inotype(u_int type)
 {
        switch(type & S_IFMT) {
@@ -321,7 +317,7 @@ inotype(u_int type)
        /* NOTREACHED */
 }
 
-static char *
+static const char *
 ftype(u_int type)
 {
        switch(type) {
index ba3534b..9a1382b 100644 (file)
 #define        INDENTNAMELEN   15
 #define        MAXLINELEN      80
 
-extern long int crc_total;
-extern int ftsoptions;
-extern int dflag, iflag, nflag, sflag;
-extern u_int keys;
-extern char fullpath[MAXPATHLEN];
-extern int lineno;
-
 static gid_t gid;
 static uid_t uid;
 static mode_t mode;
@@ -87,17 +80,17 @@ cwalk(void)
 {
        FTS *t;
        FTSENT *p;
-       time_t clock;
-       char *argv[2], host[MAXHOSTNAMELEN];
+       time_t clk;
+       char *argv[2], host[MAXHOSTNAMELEN], dot[] = ".";
        int indent = 0;
 
-       time(&clock);
+       time(&clk);
        gethostname(host, sizeof(host));
        printf(
            "#\t   user: %s\n#\tmachine: %s\n#\t   tree: %s\n#\t   date: %s",
-           getlogin(), host, fullpath, ctime(&clock));
+           getlogin(), host, fullpath, ctime(&clk));
 
-       argv[0] = ".";
+       argv[0] = dot;
        argv[1] = NULL;
        if ((t = fts_open(argv, ftsoptions, dsort)) == NULL)
                err(1, "line %d: fts_open", lineno);
index 13e540a..1337a17 100644 (file)
@@ -104,8 +104,8 @@ check_excludes(const char *fname, const char *path)
 #define MATCH(g, n) (fnmatch((g), (n), FNM_PATHNAME) == 0)
 
        LIST_FOREACH(e, &excludes, link) {
-               if (e->pathname && MATCH(e->glob, path) 
-                   || MATCH(e->glob, fname))
+               if ((e->pathname && MATCH(e->glob, path)) ||
+                   MATCH(e->glob, fname))
                        return 1;
        }
        return 0;
index 21ff599..05992fc 100644 (file)
  * $DragonFly: src/usr.sbin/mtree/extern.h,v 1.3 2003/11/03 19:31:39 eirikn Exp $
  */
 
-int     compare(char *, NODE *, FTSENT *);
+#ifdef MAXPATHLEN
+extern char fullpath[MAXPATHLEN];
+#endif
+extern int dflag, eflag, iflag, nflag, qflag, rflag, sflag, uflag;
+extern int ftsoptions;
+extern int lineno;
+extern long int crc_total;
+extern u_int keys;
+
+int     compare(NODE *, FTSENT *);
 int     crc(int, u_long *, u_long *);
 void    cwalk(void);
 char   *flags_to_string(u_long);
 
-char   *inotype(u_int);
+const char *inotype(u_int);
 u_int   parsekey(char *, int *);
 char   *rlink(char *);
 NODE   *spec(void);
index 515ae0c..9e7ffdc 100644 (file)
 #include "mtree.h"
 #include "extern.h"
 
-extern int lineno;
-
 typedef struct _key {
-       char *name;                     /* key name */
+       const char *name;               /* key name */
        u_int val;                      /* value */
 
 #define        NEEDVALUE       0x01
@@ -81,11 +79,12 @@ static KEY keylist[] = {
        {"uname",       F_UNAME,        NEEDVALUE},
 };
 
+int keycompare(const void *, const void *);
+
 u_int
 parsekey(char *name, int *needvaluep)
 {
        KEY *k, tmp;
-       int keycompare(const void *, const void *);
 
        tmp.name = name;
        k = (KEY *)bsearch(&tmp, keylist, sizeof(keylist) / sizeof(KEY),
@@ -102,7 +101,7 @@ int
 keycompare(const void *a, const void *b)
 {
 
-       return (strcmp(((KEY *)a)->name, ((KEY *)b)->name));
+       return (strcmp(((const KEY *)a)->name, ((const KEY *)b)->name));
 }
 
 char *
index 740ca29..6edcdc8 100644 (file)
@@ -46,8 +46,6 @@
 #include "mtree.h"
 #include "extern.h"
 
-extern long int crc_total;
-
 int ftsoptions = FTS_PHYSICAL;
 int cflag, dflag, eflag, iflag, nflag, qflag, rflag, sflag, uflag, Uflag;
 u_int keys;
index 5e0dfe9..d4cef4f 100644 (file)
 #include "mtree.h"
 #include "extern.h"
 
-extern long int crc_total;
-extern int ftsoptions;
-extern int dflag, eflag, qflag, rflag, sflag, uflag;
-extern char fullpath[MAXPATHLEN];
-extern int lineno;
-
 static NODE *root;
 static char path[MAXPATHLEN];
 
@@ -77,9 +71,9 @@ vwalk(void)
        FTSENT *p;
        NODE *ep, *level;
        int specdepth, rval;
-       char *argv[2];
+       char *argv[2], dot[] = ".";
 
-       argv[0] = ".";
+       argv[0] = dot;
        argv[1] = NULL;
        if ((t = fts_open(argv, ftsoptions, NULL)) == NULL)
                err(1, "line %d: fts_open", lineno);
@@ -119,7 +113,7 @@ vwalk(void)
                            !strcmp(ep->name, p->fts_name)) {
                                ep->flags |= F_VISIT;
                                if ((ep->flags & F_NOCHANGE) == 0 &&
-                                   compare(ep->name, ep, p))
+                                   compare(ep, p))
                                        rval = MISMATCHEXIT;
                                if (ep->flags & F_IGN)
                                        fts_set(t, p, FTS_SKIP);