Clean up:
authorChris Pressey <cpressey@dragonflybsd.org>
Mon, 14 Feb 2005 00:39:04 +0000 (00:39 +0000)
committerChris Pressey <cpressey@dragonflybsd.org>
Mon, 14 Feb 2005 00:39:04 +0000 (00:39 +0000)
- Remove casts to void when ignoring return values.
- Use explicit cast when calling fts_open().
- Raise WARNS to 6.

Submitted-by: Alexander 'alxl' Lobachov <alxl_at_alxl_dot_info>
usr.bin/find/Makefile
usr.bin/find/find.c
usr.bin/find/function.c
usr.bin/find/ls.c
usr.bin/find/main.c
usr.bin/find/misc.c

index e16a345..ddd7f62 100644 (file)
@@ -1,12 +1,12 @@
 #      @(#)Makefile    8.1 (Berkeley) 6/6/93
 # $FreeBSD: src/usr.bin/find/Makefile,v 1.17 2002/04/15 19:27:41 obrien Exp $
-# $DragonFly: src/usr.bin/find/Makefile,v 1.3 2005/02/13 23:49:53 cpressey Exp $
+# $DragonFly: src/usr.bin/find/Makefile,v 1.4 2005/02/14 00:39:04 cpressey Exp $
 
 PROG=  find
 SRCS=  find.c function.c ls.c main.c misc.c operator.c option.c \
        getdate.y
-WARNS?=2
-CFLAGS+= -DHAVE_SYS_TIMEB_H -I${.CURDIR}
+WARNS?=        6
+CFLAGS+=-DHAVE_SYS_TIMEB_H -I${.CURDIR}
 YFLAGS=
 
 .include <bsd.prog.mk>
index 0e79ce0..4c5af0b 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)find.c  8.5 (Berkeley) 8/5/94
  * $FreeBSD: src/usr.bin/find/find.c,v 1.17 2004/05/28 17:17:15 eik Exp $
- * $DragonFly: src/usr.bin/find/find.c,v 1.4 2005/02/13 23:49:53 cpressey Exp $
+ * $DragonFly: src/usr.bin/find/find.c,v 1.5 2005/02/14 00:39:04 cpressey Exp $
  */
 
 #include <sys/types.h>
@@ -175,7 +175,9 @@ find_execute(PLAN *plan, char *paths[])
        PLAN *p;
        int rval;
 
-       tree = fts_open(paths, ftsoptions, (issort ? find_compare : NULL));
+       tree = fts_open(paths, ftsoptions,
+           (int (*)(const FTSENT **, const FTSENT **))
+           (issort ? find_compare : NULL));
        if (tree == NULL)
                err(1, "ftsopen");
 
@@ -197,7 +199,7 @@ find_execute(PLAN *plan, char *paths[])
                case FTS_DNR:
                case FTS_ERR:
                case FTS_NS:
-                       (void)fflush(stdout);
+                       fflush(stdout);
                        warnx("%s: %s",
                            entry->fts_path, strerror(entry->fts_errno));
                        rval = 1;
@@ -209,7 +211,7 @@ find_execute(PLAN *plan, char *paths[])
                }
 #define        BADCH   " \t\n\\'\""
                if (isxargs && strpbrk(entry->fts_path, BADCH)) {
-                       (void)fflush(stdout);
+                       fflush(stdout);
                        warnx("%s: illegal path", entry->fts_path);
                        rval = 1;
                        continue;
index 3bb8177..690f3da 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)function.c  8.10 (Berkeley) 5/4/95
  * $FreeBSD: src/usr.bin/find/function.c,v 1.52 2004/07/29 03:33:55 tjr Exp $
- * $DragonFly: src/usr.bin/find/function.c,v 1.5 2005/02/13 23:49:53 cpressey Exp $
+ * $DragonFly: src/usr.bin/find/function.c,v 1.6 2005/02/14 00:39:04 cpressey Exp $
  */
 
 #include <sys/param.h>
@@ -1158,7 +1158,7 @@ c_perm(OPTION *option, char ***argvp)
 int
 f_print(PLAN *plan __unused, FTSENT *entry)
 {
-       (void)puts(entry->fts_path);
+       puts(entry->fts_path);
        return 1;
 }
 
index d649d29..1e3c982 100644 (file)
@@ -32,7 +32,7 @@
  *
  * @(#)ls.c    8.1 (Berkeley) 6/6/93
  * $FreeBSD: src/usr.bin/find/ls.c,v 1.17 2004/01/20 09:27:03 des Exp $
- * $DragonFly: src/usr.bin/find/ls.c,v 1.4 2005/02/13 23:49:53 cpressey Exp $
+ * $DragonFly: src/usr.bin/find/ls.c,v 1.5 2005/02/14 00:39:04 cpressey Exp $
  */
 
 #include <sys/param.h>
@@ -46,6 +46,7 @@
 #include <langinfo.h>
 #include <pwd.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <time.h>
 #include <unistd.h>
@@ -62,22 +63,22 @@ printlong(char *name, char *accpath, struct stat *sb)
 {
        char modep[15];
 
-       (void)printf("%6lu %8"PRId64" ", (u_long) sb->st_ino, sb->st_blocks);
-       (void)strmode(sb->st_mode, modep);
-       (void)printf("%s %3u %-*s %-*s ", modep, sb->st_nlink, MAXLOGNAME - 1,
+       printf("%6lu %8"PRId64" ", (u_long) sb->st_ino, sb->st_blocks);
+       strmode(sb->st_mode, modep);
+       printf("%s %3u %-*s %-*s ", modep, sb->st_nlink, MAXLOGNAME - 1,
            user_from_uid(sb->st_uid, 0), MAXLOGNAME - 1,
            group_from_gid(sb->st_gid, 0));
 
        if (S_ISCHR(sb->st_mode) || S_ISBLK(sb->st_mode))
-               (void)printf("%3d, %3d ", major(sb->st_rdev),
+               printf("%3d, %3d ", major(sb->st_rdev),
                    minor(sb->st_rdev));
        else
-               (void)printf("%8"PRId64" ", sb->st_size);
+               printf("%8"PRId64" ", sb->st_size);
        printtime(sb->st_mtime);
-       (void)printf("%s", name);
+       printf("%s", name);
        if (S_ISLNK(sb->st_mode))
                printlink(accpath);
-       (void)putchar('\n');
+       putchar('\n');
 }
 
 static void
@@ -115,5 +116,5 @@ printlink(char *name)
                return;
        }
        path[lnklen] = '\0';
-       (void)printf(" -> %s", path);
+       printf(" -> %s", path);
 }
index d30250e..173efd8 100644 (file)
@@ -36,7 +36,7 @@
  * @(#) Copyright (c) 1990, 1993, 1994 The Regents of the University of California.  All rights reserved.
  * @(#)main.c  8.4 (Berkeley) 5/4/95
  * $FreeBSD: src/usr.bin/find/main.c,v 1.15 2003/06/14 13:00:21 markm Exp $
- * $DragonFly: src/usr.bin/find/main.c,v 1.4 2005/02/13 23:49:53 cpressey Exp $
+ * $DragonFly: src/usr.bin/find/main.c,v 1.5 2005/02/14 00:39:04 cpressey Exp $
  */
 
 #include <sys/types.h>
@@ -74,9 +74,9 @@ main(int argc, char *argv[])
        char **p, **start;
        int Hflag, Lflag, ch;
 
-       (void)setlocale(LC_ALL, "");
+       setlocale(LC_ALL, "");
 
-       (void)time(&now);       /* initialize the time-of-day */
+       time(&now);     /* initialize the time-of-day */
 
        p = start = argv;
        Hflag = Lflag = 0;
@@ -153,7 +153,7 @@ main(int argc, char *argv[])
 static void
 usage(void)
 {
-       (void)fprintf(stderr,
+       fprintf(stderr,
 "usage: find [-H | -L | -P] [-EXdsx] [-f file] [file ...] [expression]\n");
        exit(1);
 }
index 7439b06..17955fc 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)misc.c  8.2 (Berkeley) 4/1/94
  * $FreeBSD: src/usr.bin/find/misc.c,v 1.7 2003/06/14 13:00:21 markm Exp $
- * $DragonFly: src/usr.bin/find/misc.c,v 1.4 2005/02/13 23:49:53 cpressey Exp $
+ * $DragonFly: src/usr.bin/find/misc.c,v 1.5 2005/02/14 00:39:04 cpressey Exp $
  */
 
 #include <sys/types.h>
@@ -84,11 +84,11 @@ queryuser(char *argv[])
 {
        int ch, first, nl;
 
-       (void)fprintf(stderr, "\"%s", *argv);
+       fprintf(stderr, "\"%s", *argv);
        while (*++argv)
-               (void)fprintf(stderr, " %s", *argv);
-       (void)fprintf(stderr, "\"? ");
-       (void)fflush(stderr);
+               fprintf(stderr, " %s", *argv);
+       fprintf(stderr, "\"? ");
+       fflush(stderr);
 
        first = ch = getchar();
        for (nl = 0;;) {
@@ -102,8 +102,8 @@ queryuser(char *argv[])
        }
 
        if (!nl) {
-               (void)fprintf(stderr, "\n");
-               (void)fflush(stderr);
+               fprintf(stderr, "\n");
+               fflush(stderr);
        }
         return (first == 'y');
 }