WARNS6 clean.
authorMax Okumoto <okumoto@dragonflybsd.org>
Thu, 13 Jan 2005 18:57:56 +0000 (18:57 +0000)
committerMax Okumoto <okumoto@dragonflybsd.org>
Thu, 13 Jan 2005 18:57:56 +0000 (18:57 +0000)
Changes:
- removed the (void) cast of functions that don't use a
  return value;
- the envtmp declaration moved into extern.h;
- type of some pointers to constant string changed from
  'char *' to 'char const *';

13 files changed:
usr.bin/ar/Makefile
usr.bin/ar/append.c
usr.bin/ar/ar.c
usr.bin/ar/archive.c
usr.bin/ar/archive.h
usr.bin/ar/contents.c
usr.bin/ar/delete.c
usr.bin/ar/extern.h
usr.bin/ar/extract.c
usr.bin/ar/misc.c
usr.bin/ar/move.c
usr.bin/ar/print.c
usr.bin/ar/replace.c

index 636fe9d..01d336e 100644 (file)
@@ -1,6 +1,6 @@
 #      @(#)Makefile    8.1 (Berkeley) 6/6/93
 # $FreeBSD: src/usr.bin/ar/Makefile,v 1.9.2.1 2001/04/25 11:29:15 ru Exp $
-# $DragonFly: src/usr.bin/ar/Attic/Makefile,v 1.4 2004/01/28 16:41:17 joerg Exp $
+# $DragonFly: src/usr.bin/ar/Attic/Makefile,v 1.5 2005/01/13 18:57:56 okumoto Exp $
 
 PROG=  ar
 CFLAGS+=-I${.CURDIR} -Wall
@@ -8,5 +8,6 @@ SRCS=   append.c ar.c archive.c contents.c delete.c extract.c misc.c \
        move.c print.c replace.c
 MAN=   ar.1aout ar.5
 BINDIR=        /usr/libexec/binutils212/aout
+WARNS?= 6
 
 .include <bsd.prog.mk>
index a95734a..5fd0c33 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)append.c        8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/append.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/append.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -77,11 +77,11 @@ append(char **argv)
                        continue;
                }
                if (options & AR_V)
-                       (void)printf("q - %s\n", file);
+                       printf("q - %s\n", file);
                cf.rfd = fd;
                cf.rname = file;
                put_arobj(&cf, &sb);
-               (void)close(fd);
+               close(fd);
        }
        close_archive(afd);
        return (eval);
index fb66f93..b9a088d 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/ar/ar.c,v 1.8.2.1 2001/08/02 00:51:00 obrien Exp $
- * $DragonFly: src/usr.bin/ar/Attic/ar.c,v 1.4 2003/11/03 19:31:28 eirikn Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/ar.c,v 1.5 2005/01/13 18:57:56 okumoto Exp $
  *
  * @(#) Copyright (c) 1990, 1993, 1994 The Regents of the University of California.  All rights reserved.
  * @(#)ar.c    8.3 (Berkeley) 4/2/94
@@ -58,8 +58,9 @@
 
 CHDR chdr;
 u_int options;
-char *archive, *envtmp, *posarg, *posname;
-static void badoptions(char *);
+char *archive, *posarg, *posname;
+char const *envtmp;
+static void badoptions(char const*);
 static void usage(void);
 
 /*
@@ -75,7 +76,7 @@ main(int argc, char **argv)
        char *p;
        int (*fcall)(char **) = NULL;
 
-       (void) setlocale(LC_TIME, "");;
+       setlocale(LC_TIME, "");;
 
        if (argc < 3)
                usage();
@@ -88,7 +89,7 @@ main(int argc, char **argv)
                if (!(p = malloc((u_int)(strlen(argv[1]) + 2))))
                        err(1, NULL);
                *p = '-';
-               (void)strcpy(p + 1, argv[1]);
+               strcpy(p + 1, argv[1]);
                argv[1] = p;
        }
 
@@ -210,7 +211,7 @@ main(int argc, char **argv)
 }
 
 static void
-badoptions(char *arg)
+badoptions(char const *arg)
 {
 
        warnx("illegal option combination for %s", arg);
@@ -221,7 +222,7 @@ static void
 usage(void)
 {
 
-       (void)fprintf(stderr, "%s\n%s\n%s\n%s\n%s\n%s\n%s\n%s\n%s\n",
+       fprintf(stderr, "%s\n%s\n%s\n%s\n%s\n%s\n%s\n%s\n%s\n",
                "usage: ar -d [-Tv] archive file ...",
                "       ar -m [-Tv] archive file ...",
                "       ar -m [-abiTv] position archive file ...",
index 986139a..9520624 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)archive.c       8.3 (Berkeley) 4/2/94
  * $FreeBSD: src/usr.bin/ar/archive.c,v 1.10.6.1 2001/08/02 00:51:00 obrien Exp $
- * $DragonFly: src/usr.bin/ar/Attic/archive.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/archive.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -110,7 +110,7 @@ void
 close_archive(int fd)
 {
 
-       (void)close(fd);                        /* Implicit unlock. */
+       close(fd);                      /* Implicit unlock. */
 }
 
 /* Convert ar header field to an integer. */
@@ -207,7 +207,7 @@ put_arobj(CF *cfp, struct stat *sb)
         */
        if (sb) {
                name = basename(cfp->rname);
-               (void)fstat(cfp->rfd, sb);
+               fstat(cfp->rfd, sb);
 
                /*
                 * If not truncating names and the name is too long or contains
@@ -216,23 +216,23 @@ put_arobj(CF *cfp, struct stat *sb)
                lname = strlen(name);
                if (options & AR_TR) {
                        if (lname > OLDARMAXNAME) {
-                               (void)fflush(stdout);
+                               fflush(stdout);
                                warnx("warning: %s truncated to %.*s",
                                    name, OLDARMAXNAME, name);
-                               (void)fflush(stderr);
+                               fflush(stderr);
                        }
-                       (void)sprintf(hb, HDR3, name,
+                       sprintf(hb, HDR3, name,
                            (long)sb->st_mtimespec.tv_sec, sb->st_uid,
                            sb->st_gid, sb->st_mode, sb->st_size, ARFMAG);
                        lname = 0;
-               } else if (lname > sizeof(hdr->ar_name) || strchr(name, ' '))
-                       (void)sprintf(hb, HDR1, AR_EFMT1, lname,
+               } else if (lname > (long)sizeof(hdr->ar_name) || strchr(name, ' '))
+                       sprintf(hb, HDR1, AR_EFMT1, lname,
                            (long)sb->st_mtimespec.tv_sec, sb->st_uid,
                            sb->st_gid, sb->st_mode, sb->st_size + lname,
                            ARFMAG);
                else {
                        lname = 0;
-                       (void)sprintf(hb, HDR2, name,
+                       sprintf(hb, HDR2, name,
                            (long)sb->st_mtimespec.tv_sec, sb->st_uid,
                            sb->st_gid, sb->st_mode, sb->st_size, ARFMAG);
                }
index d153c3b..30ca4ef 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  *     @(#)archive.h   8.3 (Berkeley) 4/2/94
- * $DragonFly: src/usr.bin/ar/Attic/archive.h,v 1.3 2003/11/03 19:31:28 eirikn Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/archive.h,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 /* Ar(1) options. */
@@ -66,9 +66,9 @@ extern u_int options;
 /* File copy structure. */
 typedef struct {
        int rfd;                        /* read file descriptor */
-       char *rname;                    /* read name */
+       char const *rname;                      /* read name */
        int wfd;                        /* write file descriptor */
-       char *wname;                    /* write name */
+       char const *wname;                      /* write name */
 #define        NOPAD   0x00                    /* don't pad */
 #define        RPAD    0x01                    /* pad on reads */
 #define        WPAD    0x02                    /* pad on writes */
index a0d92e2..539a0f8 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/ar/contents.c,v 1.5.6.2 2002/03/12 19:33:00 phantom Exp $
- * $DragonFly: src/usr.bin/ar/Attic/contents.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/contents.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  *
  * @(#)contents.c      8.3 (Berkeley) 4/2/94
  * $FreeBSD: src/usr.bin/ar/contents.c,v 1.5.6.2 2002/03/12 19:33:00 phantom Exp $
@@ -77,20 +77,20 @@ contents(char **argv)
                if (options & AR_V) {
                        static int d_first = -1;
 
-                       (void)strmode(chdr.mode, buf);
-                       (void)printf("%s %6d/%-6d %8qd ",
+                       strmode(chdr.mode, buf);
+                       printf("%s %6d/%-6d %8qd ",
                            buf + 1, chdr.uid, chdr.gid, chdr.size);
 
                        if (d_first < 0)
                                d_first = (*nl_langinfo(D_MD_ORDER) == 'd');
                        tp = localtime(&chdr.date);
-                       (void)strftime(buf, sizeof(buf),
+                       strftime(buf, sizeof(buf),
                                       d_first ? "%e %b %R %Y" :
                                                 "%b %e %R %Y",
                                       tp);
-                       (void)printf("%s %s\n", buf, file);
+                       printf("%s %s\n", buf, file);
                } else
-                       (void)printf("%s\n", file);
+                       printf("%s\n", file);
                if (!all && !*argv)
                        break;
 next:          skip_arobj(afd);
index db45334..76e0e80 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)delete.c        8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/delete.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/delete.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -71,7 +71,7 @@ delete(char **argv)
        while (get_arobj(afd)) {
                if (*argv && (file = files(argv))) {
                        if (options & AR_V)
-                               (void)printf("d - %s\n", file);
+                               printf("d - %s\n", file);
                        skip_arobj(afd);
                        continue;
                }
@@ -79,12 +79,12 @@ delete(char **argv)
        }
 
        size = lseek(tfd, (off_t)0, SEEK_CUR);
-       (void)lseek(tfd, (off_t)0, SEEK_SET);
-       (void)lseek(afd, (off_t)SARMAG, SEEK_SET);
+       lseek(tfd, (off_t)0, SEEK_SET);
+       lseek(afd, (off_t)SARMAG, SEEK_SET);
        SETCF(tfd, tname, afd, archive, NOPAD);
        copy_ar(&cf, size);
-       (void)close(tfd);
-       (void)ftruncate(afd, size + SARMAG);
+       close(tfd);
+       ftruncate(afd, size + SARMAG);
        close_archive(afd);
 
        if (*argv) {
index cfa716c..7608de6 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)extern.h    8.3 (Berkeley) 4/2/94
  * $FreeBSD: src/usr.bin/ar/extern.h,v 1.1.1.1.14.1 2001/08/02 00:51:00 obrien Exp $
- * $DragonFly: src/usr.bin/ar/Attic/extern.h,v 1.3 2003/11/03 19:31:28 eirikn Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/extern.h,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 int    append(char **);
@@ -40,7 +40,7 @@ void  badfmt(void);
 int    compare(char *);
 int    contents(char **);
 int    delete(char **);
-void   error(char *);
+void   error(char const*);
 int    extract(char **);
 char   *files(char **argv);
 int    move(char **);
@@ -51,5 +51,6 @@ int   tmp(void);
 
 extern char *archive;
 extern char *posarg, *posname;         /* positioning file name */
-extern char *tname;                     /* temporary file "name" */
+extern char const *tname;           /* temporary file "name" */
 extern CHDR chdr;                      /* converted header */
+extern char const *envtmp;
index 974de16..29a357a 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)extract.c       8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/extract.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/extract.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -96,7 +96,7 @@ extract(char **argv)
                }
 
                if (options & AR_V)
-                       (void)printf("x - %s\n", file);
+                       printf("x - %s\n", file);
 
                cf.wfd = tfd;
                cf.wname = file;
@@ -113,7 +113,7 @@ extract(char **argv)
                                eval = 1;
                        }
                }
-               (void)close(tfd);
+               close(tfd);
                if (!all && !*argv)
                        break;
        }
index 6310d23..486570c 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)misc.c  8.3 (Berkeley) 4/2/94
  * $FreeBSD: src/usr.bin/ar/misc.c,v 1.6.6.1 2001/08/02 00:51:00 obrien Exp $
- * $DragonFly: src/usr.bin/ar/Attic/misc.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/misc.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
 #include "extern.h"
 #include "pathnames.h"
 
-char *tname = "temporary file";                /* temporary file "name" */
+char const *tname = "temporary file";          /* temporary file "name" */
 
 int
 tmp(void)
 {
-       extern char *envtmp;
        sigset_t set, oset;
        static int first;
        int fd;
@@ -71,16 +70,16 @@ tmp(void)
        }
 
        if (envtmp)
-               (void)sprintf(path, "%s/%s", envtmp, _NAME_ARTMP);
+               sprintf(path, "%s/%s", envtmp, _NAME_ARTMP);
        else
                strcpy(path, _PATH_ARTMP);
 
        sigfillset(&set);
-       (void)sigprocmask(SIG_BLOCK, &set, &oset);
+       sigprocmask(SIG_BLOCK, &set, &oset);
        if ((fd = mkstemp(path)) == -1)
                error(tname);
-        (void)unlink(path);
-       (void)sigprocmask(SIG_SETMASK, &oset, NULL);
+        unlink(path);
+       sigprocmask(SIG_SETMASK, &oset, NULL);
        return (fd);
 }
 
@@ -127,7 +126,7 @@ badfmt(void)
 }
 
 void
-error(char *name)
+error(char const *name)
 {
 
        err(1, "%s", name);
index 9361d7b..13b2b1a 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)move.c  8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/move.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/move.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -87,7 +87,7 @@ move(char **argv)
        for (curfd = tfd1; get_arobj(afd);) {
                if (*argv && (file = files(argv))) {
                        if (options & AR_V)
-                               (void)printf("m - %s\n", file);
+                               printf("m - %s\n", file);
                        cf.wfd = tfd2;
                        put_arobj(&cf, (struct stat *)NULL);
                        continue;
@@ -111,24 +111,24 @@ move(char **argv)
                close_archive(afd);
                return (1);
        }
-       (void)lseek(afd, (off_t)SARMAG, SEEK_SET);
+       lseek(afd, (off_t)SARMAG, SEEK_SET);
 
        SETCF(tfd1, tname, afd, archive, NOPAD);
        tsize = size = lseek(tfd1, (off_t)0, SEEK_CUR);
-       (void)lseek(tfd1, (off_t)0, SEEK_SET);
+       lseek(tfd1, (off_t)0, SEEK_SET);
        copy_ar(&cf, size);
 
        tsize += size = lseek(tfd2, (off_t)0, SEEK_CUR);
-       (void)lseek(tfd2, (off_t)0, SEEK_SET);
+       lseek(tfd2, (off_t)0, SEEK_SET);
        cf.rfd = tfd2;
        copy_ar(&cf, size);
 
        tsize += size = lseek(tfd3, (off_t)0, SEEK_CUR);
-       (void)lseek(tfd3, (off_t)0, SEEK_SET);
+       lseek(tfd3, (off_t)0, SEEK_SET);
        cf.rfd = tfd3;
        copy_ar(&cf, size);
 
-       (void)ftruncate(afd, tsize + SARMAG);
+       ftruncate(afd, tsize + SARMAG);
        close_archive(afd);
 
        if (*argv) {
index d667961..1251426 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)print.c 8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/print.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/print.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -72,8 +72,8 @@ print(char **argv)
                        continue;
                }
                if (options & AR_V) {
-                       (void)printf("\n<%s>\n\n", file);
-                       (void)fflush(stdout);
+                       printf("\n<%s>\n\n", file);
+                       fflush(stdout);
                }
                copy_ar(&cf, chdr.size);
                if (!all && !*argv)
index d90b9af..2dbb2d2 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)replace.c       8.3 (Berkeley) 4/2/94
  *
- * $DragonFly: src/usr.bin/ar/Attic/replace.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/ar/Attic/replace.c,v 1.4 2005/01/13 18:57:56 okumoto Exp $
  */
 
 #include <sys/param.h>
@@ -100,19 +100,19 @@ replace(char **argv)
                                warn("%s", file);
                                goto useold;
                        }
-                       (void)fstat(sfd, &sb);
+                       fstat(sfd, &sb);
                        if (options & AR_U && sb.st_mtime <= chdr.date) {
-                               (void) close(sfd);
+                               close(sfd);
                                goto useold;
                        }
 
                        if (options & AR_V)
-                            (void)printf("r - %s\n", file);
+                            printf("r - %s\n", file);
 
                        /* Read from disk, write to an archive; pad on write */
                        SETCF(sfd, file, curfd, tname, WPAD);
                        put_arobj(&cf, &sb);
-                       (void)close(sfd);
+                       close(sfd);
                        skip_arobj(afd);
                        continue;
                }
@@ -142,36 +142,36 @@ useold:                   SETCF(afd, archive, curfd, tname, RPAD|WPAD);
        /* Append any left-over arguments to the end of the after file. */
 append:        while ( (file = *argv++) ) {
                if (options & AR_V)
-                       (void)printf("a - %s\n", file);
+                       printf("a - %s\n", file);
                if ((sfd = open(file, O_RDONLY)) < 0) {
                        errflg = 1;
                        warn("%s", file);
                        continue;
                }
-               (void)fstat(sfd, &sb);
+               fstat(sfd, &sb);
                /* Read from disk, write to an archive; pad on write. */
                SETCF(sfd, file,
                    options & (AR_A|AR_B) ? tfd1 : tfd2, tname, WPAD);
                put_arobj(&cf, &sb);
-               (void)close(sfd);
+               close(sfd);
        }
 
-       (void)lseek(afd, (off_t)SARMAG, SEEK_SET);
+       lseek(afd, (off_t)SARMAG, SEEK_SET);
 
        SETCF(tfd1, tname, afd, archive, NOPAD);
        if (tfd1 != -1) {
                tsize = size = lseek(tfd1, (off_t)0, SEEK_CUR);
-               (void)lseek(tfd1, (off_t)0, SEEK_SET);
+               lseek(tfd1, (off_t)0, SEEK_SET);
                copy_ar(&cf, size);
        } else
                tsize = 0;
 
        tsize += size = lseek(tfd2, (off_t)0, SEEK_CUR);
-       (void)lseek(tfd2, (off_t)0, SEEK_SET);
+       lseek(tfd2, (off_t)0, SEEK_SET);
        cf.rfd = tfd2;
        copy_ar(&cf, size);
 
-       (void)ftruncate(afd, tsize + SARMAG);
+       ftruncate(afd, tsize + SARMAG);
        close_archive(afd);
        return (errflg);
 }