Cleanup:
authorSascha Wildner <swildner@dragonflybsd.org>
Sun, 6 Nov 2005 11:44:02 +0000 (11:44 +0000)
committerSascha Wildner <swildner@dragonflybsd.org>
Sun, 6 Nov 2005 11:44:02 +0000 (11:44 +0000)
- In function definitions, move the type on a line of its own.
- Ansify function definitions.
- Remove (void) casts for discarded return values.

bin/df/df.c
bin/expr/expr.y
bin/ls/print.c
bin/mined/mined1.c
bin/mined/mined2.c
bin/sh/histedit.c
bin/sh/parser.c

index 67f0e1f..9dcc61c 100644 (file)
@@ -38,7 +38,7 @@
  * @(#) Copyright (c) 1980, 1990, 1993, 1994 The Regents of the University of California.  All rights reserved.
  * @(#)df.c    8.9 (Berkeley) 5/8/95
  * $FreeBSD: src/bin/df/df.c,v 1.23.2.9 2002/07/01 00:14:24 iedowse Exp $
- * $DragonFly: src/bin/df/df.c,v 1.5 2004/11/07 20:54:51 eirikn Exp $
+ * $DragonFly: src/bin/df/df.c,v 1.6 2005/11/06 11:44:02 swildner Exp $
  */
 
 #include <sys/cdefs.h>
@@ -117,7 +117,8 @@ void          usage(void);
 int    aflag = 0, hflag, iflag, nflag;
 struct ufs_args mdev;
 
-static __inline int imax(int a, int b)
+static __inline int
+imax(int a, int b)
 {
        return (a > b ? a : b);
 }
index b84b993..0bcd6e0 100644 (file)
@@ -5,7 +5,7 @@
  * Largely rewritten by J.T. Conklin (jtc@wimsey.com)
  *
  * $FreeBSD: src/bin/expr/expr.y,v 1.14.2.3 2001/08/01 02:37:46 obrien Exp $
- * $DragonFly: src/bin/expr/expr.y,v 1.5 2004/11/07 20:54:51 eirikn Exp $
+ * $DragonFly: src/bin/expr/expr.y,v 1.6 2005/11/06 11:44:02 swildner Exp $
  */
 
 #include <sys/types.h>
@@ -107,8 +107,7 @@ expr:       TOKEN
 %%
 
 struct val *
-make_integer (i)
-quad_t i;
+make_integer(quad_t i)
 {
        struct val *vp;
 
@@ -123,8 +122,7 @@ quad_t i;
 }
 
 struct val *
-make_str (s)
-const char *s;
+make_str(const char *s)
 {
        struct val *vp;
        size_t i;
@@ -153,8 +151,7 @@ const char *s;
 
 
 void
-free_value (vp)
-struct val *vp;
+free_value(struct val *vp)
 {
        if (vp->type == string || vp->type == numeric_string)
                free (vp->u.s); 
@@ -162,8 +159,7 @@ struct val *vp;
 
 
 quad_t
-to_integer (vp)
-struct val *vp;
+to_integer(struct val *vp)
 {
        quad_t i;
 
@@ -186,8 +182,7 @@ struct val *vp;
 }
 
 void
-to_string (vp)
-struct val *vp;
+to_string(struct val *vp)
 {
        char *tmp;
 
@@ -206,8 +201,7 @@ struct val *vp;
 
 
 int
-isstring (vp)
-struct val *vp;
+isstring(struct val *vp)
 {
        /* only TRUE if this string is not a valid integer */
        return (vp->type == string);
@@ -215,7 +209,7 @@ struct val *vp;
 
 
 int
-yylex ()
+yylex(void)
 {
        char *p;
 
@@ -240,8 +234,7 @@ yylex ()
 }
 
 int
-is_zero_or_null (vp)
-struct val *vp;
+is_zero_or_null(struct val *vp)
 {
        if (vp->type == integer) {
                return (vp->u.i == 0);
@@ -252,9 +245,7 @@ struct val *vp;
 }
 
 int
-main (argc, argv)
-int argc __unused;
-char **argv;
+main (int argc __unused, char **argv)
 {
        setlocale (LC_ALL, "");
 
@@ -271,16 +262,14 @@ char **argv;
 }
 
 int
-yyerror (s)
-const char *s __unused;
+yyerror(const char *s __unused)
 {
        errx (2, "syntax error");
 }
 
 
 struct val *
-op_or (a, b)
-struct val *a, *b;
+op_or(struct val *a, struct val *b)
 {
        if (is_zero_or_null (a)) {
                free_value (a);
@@ -292,8 +281,7 @@ struct val *a, *b;
 }
                
 struct val *
-op_and (a, b)
-struct val *a, *b;
+op_and(struct val *a, struct val *b)
 {
        if (is_zero_or_null (a) || is_zero_or_null (b)) {
                free_value (a);
@@ -306,8 +294,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_eq (a, b)
-struct val *a, *b;
+op_eq(struct val *a, struct val *b)
 {
        struct val *r; 
 
@@ -327,8 +314,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_gt (a, b)
-struct val *a, *b;
+op_gt(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -348,8 +334,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_lt (a, b)
-struct val *a, *b;
+op_lt(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -369,8 +354,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_ge (a, b)
-struct val *a, *b;
+op_ge(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -390,8 +374,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_le (a, b)
-struct val *a, *b;
+op_le(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -411,8 +394,7 @@ struct val *a, *b;
 }
 
 struct val *
-op_ne (a, b)
-struct val *a, *b;
+op_ne(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -432,8 +414,7 @@ struct val *a, *b;
 }
 
 int
-chk_plus (a, b, r)
-quad_t a, b, r;
+chk_plus(quad_t a, quad_t b, quad_t r)
 {
        /* sum of two positive numbers must be positive */
        if (a > 0 && b > 0 && r <= 0)
@@ -446,8 +427,7 @@ quad_t a, b, r;
 }
 
 struct val *
-op_plus (a, b)
-struct val *a, *b;
+op_plus(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -465,8 +445,7 @@ struct val *a, *b;
 }
 
 int
-chk_minus (a, b, r)
-quad_t a, b, r;
+chk_minus(quad_t a, quad_t b, quad_t r)
 {
        /* special case subtraction of QUAD_MIN */
        if (b == QUAD_MIN) {
@@ -480,8 +459,7 @@ quad_t a, b, r;
 }
 
 struct val *
-op_minus (a, b)
-struct val *a, *b;
+op_minus(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -499,8 +477,7 @@ struct val *a, *b;
 }
 
 int
-chk_times (a, b, r)
-quad_t a, b, r;
+chk_times(quad_t a, quad_t b, quad_t r)
 {
        /* special case: first operand is 0, no overflow possible */
        if (a == 0)
@@ -512,8 +489,7 @@ quad_t a, b, r;
 }
 
 struct val *
-op_times (a, b)
-struct val *a, *b;
+op_times(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -531,8 +507,7 @@ struct val *a, *b;
 }
 
 int
-chk_div (a, b)
-quad_t a, b;
+chk_div(quad_t a, quad_t b)
 {
        /* div by zero has been taken care of before */
        /* only QUAD_MIN / -1 causes overflow */
@@ -543,8 +518,7 @@ quad_t a, b;
 }
 
 struct val *
-op_div (a, b)
-struct val *a, *b;
+op_div(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -566,8 +540,7 @@ struct val *a, *b;
 }
        
 struct val *
-op_rem (a, b)
-struct val *a, *b;
+op_rem(struct val *a, struct val *b)
 {
        struct val *r;
 
@@ -587,8 +560,7 @@ struct val *a, *b;
 }
        
 struct val *
-op_colon (a, b)
-struct val *a, *b;
+op_colon(struct val *a, struct val *b)
 {
        regex_t rp;
        regmatch_t rm[2];
index ec6500e..8dfb683 100644 (file)
@@ -31,7 +31,7 @@
  *
  * @(#)print.c 8.4 (Berkeley) 4/17/94
  * $FreeBSD: src/bin/ls/print.c,v 1.73 2004/06/08 09:27:42 das Exp $
- * $DragonFly: src/bin/ls/print.c,v 1.16 2005/09/19 10:14:29 asmodai Exp $
+ * $DragonFly: src/bin/ls/print.c,v 1.17 2005/11/06 11:44:02 swildner Exp $
  */
 
 #include <sys/param.h>
@@ -586,7 +586,7 @@ printsize(size_t width, off_t bytes)
 
                humanize_number(buf, sizeof(buf), (int64_t)bytes, "",
                                HN_AUTOSCALE, HN_B | HN_NOSPACE | HN_DECIMAL);
-               (void)printf("%5s ", buf);
+               printf("%5s ", buf);
        } else
                printf("%*lld ", width, bytes);
 }
index f41480f..b3b2795 100644 (file)
@@ -33,7 +33,7 @@
  *      EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * [original code from minix codebase]
- * $DragonFly: src/bin/mined/mined1.c,v 1.7 2005/10/29 12:05:27 swildner Exp $*
+ * $DragonFly: src/bin/mined/mined1.c,v 1.8 2005/11/06 11:44:02 swildner Exp $*
  */
 /*
  * Part one of the mined editor.
@@ -544,7 +544,7 @@ WT(void)
   if (count > 0L && flush_buffer(fd) == ERRORS)
        count = -1L;
 
-  (void) close(fd);
+  close(fd);
 
   if (count == -1L)
        return ERRORS;
index 1c49ddd..3bbf4df 100644 (file)
@@ -33,7 +33,7 @@
  *      EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * [original code from minix codebase]
- * $DragonFly: src/bin/mined/mined2.c,v 1.5 2005/10/29 12:05:27 swildner Exp $*
+ * $DragonFly: src/bin/mined/mined2.c,v 1.6 2005/11/06 11:44:02 swildner Exp $*
  */
 /*
  * Part 2 of the mined editor.
@@ -655,7 +655,7 @@ insert(LINE *line, char *location, char *string)
   *bufp = '\0';
   
   if (*(string - 1) == '\n')           /* Insert a new line */
-       (void) line_insert(line, location, length_of(location));
+       line_insert(line, location, length_of(location));
   else                                 /* Append last part of line */
        copy_string(bufp, location);
 
@@ -795,7 +795,7 @@ PT(int u __unused)
        error("Buffer is empty.", NIL_PTR);
   else {
        file_insert(fd, FALSE);/* Insert the buffer */
-       (void) close(fd);
+       close(fd);
   }
 }
 
@@ -817,7 +817,7 @@ IF(int u __unused)
        error("Cannot open ", name);
   else {
        file_insert(fd, TRUE);  /* Insert the file */
-       (void) close(fd);
+       close(fd);
   }
 }
 
@@ -848,7 +848,7 @@ file_insert(int fd, FLAG old_pos)
   
   if (ret == NO_LINE) {                /* Last line read not ended by a '\n' */
        line = line->next;
-       (void) insert(line, line->text, line_buffer);
+       insert(line, line->text, line_buffer);
   }
 
 /* Calculate nr of lines added */
index 6020e41..5c246bf 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)histedit.c      8.2 (Berkeley) 5/4/95
  * $FreeBSD: src/bin/sh/histedit.c,v 1.13.2.4 2002/08/27 01:36:28 tjr Exp $
- * $DragonFly: src/bin/sh/histedit.c,v 1.4 2004/03/19 18:39:41 cpressey Exp $
+ * $DragonFly: src/bin/sh/histedit.c,v 1.5 2005/11/06 11:44:02 swildner Exp $
  */
 
 #include <sys/param.h>
@@ -144,8 +144,7 @@ bad:
 
 
 void
-sethistsize(hs)
-       const char *hs;
+sethistsize(const char *hs)
 {
        int histsize;
 
index 2aed73c..345c816 100644 (file)
@@ -35,7 +35,7 @@
  *
  * @(#)parser.c        8.7 (Berkeley) 5/16/95
  * $FreeBSD: src/bin/sh/parser.c,v 1.29.2.9 2002/10/18 11:24:04 tjr Exp $
- * $DragonFly: src/bin/sh/parser.c,v 1.5 2004/10/01 20:30:25 dillon Exp $
+ * $DragonFly: src/bin/sh/parser.c,v 1.6 2005/11/06 11:44:02 swildner Exp $
  */
 
 #include <stdlib.h>
@@ -614,7 +614,8 @@ makename(void)
        return n;
 }
 
-void fixredir(union node *n, const char *text, int err)
+void
+fixredir(union node *n, const char *text, int err)
 {
        TRACE(("Fix redir %s %d\n", text, err));
        if (!err)