code-style: minor cleanup and nitpicking

env.c

@@ -20,7 +20,7 @@ main(int argc, char *argv[])
 {
 	ARGBEGIN {
 	case 'i':
-		if(environ)
+		if (environ)
 			*environ = NULL;
 		break;
 	case 'u':

expr.c

@@ -80,10 +80,10 @@ doop(int *op, int **opp, Val *val, Val **valp)
 		break;
 	case '=': ret = (Val){ NULL, valcmp(a, b) == 0 }; break;
 	case '>': ret = (Val){ NULL, valcmp(a, b) >  0 }; break;
-	case GE : ret = (Val){ NULL, valcmp(a, b) >= 0 }; break;
+	case GE: ret = (Val){ NULL, valcmp(a, b) >= 0 }; break;
 	case '<': ret = (Val){ NULL, valcmp(a, b) <  0 }; break;
-	case LE : ret = (Val){ NULL, valcmp(a, b) <= 0 }; break;
-	case NE : ret = (Val){ NULL, valcmp(a, b) != 0 }; break;
+	case LE: ret = (Val){ NULL, valcmp(a, b) <= 0 }; break;
+	case NE: ret = (Val){ NULL, valcmp(a, b) != 0 }; break;
 
 	case '+': enan(a); enan(b); ret = (Val){ NULL, a.n + b.n }; break;
 	case '-': enan(a); enan(b); ret = (Val){ NULL, a.n - b.n }; break;

logger.c

@@ -69,7 +69,7 @@ main(int argc, char *argv[])
 	openlog(tag ? tag : getlogin(), logflags, 0);
 
 	if (argc == 0) {
-		while(getline(&buf, &sz, stdin) != -1)
+		while (getline(&buf, &sz, stdin) != -1)
 			syslog(priority, "%s", buf);
 		if (ferror(stdin))
 			eprintf("%s: read error:", "<stdin>");

sed.c

@@ -820,6 +820,7 @@ char *
 get_s_arg(Cmd *c, char *s)
 {
 	Rune delim, r;
+	Cmd buf;
 	char *p;
 	int esc;
 
@@ -901,13 +902,13 @@ get_s_arg(Cmd *c, char *s)
 	for (; s < p; s++) {
 		if (isdigit(*s)) {
 			c->u.s.occurrence = stol(s, &s);
-		} else switch (*s) {
-			case 'p' : c->u.s.p = 1;          break;
-			case 'g' : c->u.s.occurrence = 0; break;
-			case 'w' : {
+		} else {
+			switch (*s) {
+			case 'g': c->u.s.occurrence = 0; break;
+			case 'p': c->u.s.p = 1;          break;
+			case 'w':
 				/* must be last flag, take everything up to newline/semicolon
 				 * s == p after this */
-				Cmd buf;
 				s = get_w_arg(&buf, s);
 				c->u.s.file = buf.u.file;
 				break;
@@ -927,7 +928,6 @@ free_s_arg(Cmd *c)
 	free(c->u.s.repl.str);
 }
 
-
 /* see get_r_arg notes */
 char *
 get_w_arg(Cmd *c, char *s)
@@ -1092,13 +1092,13 @@ match_addr(Addr *a)
 	switch (a->type) {
 	default:
 	case IGNORE: return 0;
-	case EVERY : return 1;
-	case LINE  : return lineno == a->u.lineno;
-	case LAST  :
+	case EVERY: return 1;
+	case LINE: return lineno == a->u.lineno;
+	case LAST:
 		while (is_eof(file) && !next_file())
 			;
 		return !file;
-	case REGEX :
+	case REGEX:
 		lastre = a->u.re;
 		return !regexec(a->u.re, patt.str, 0, NULL, 0);
 	case LASTRE:

seq.c

@@ -27,7 +27,7 @@ main(int argc, char *argv[])
 
 	ARGBEGIN {
 	case 'f':
-		if(!validfmt(tmp=EARGF(usage())))
+		if (!validfmt(tmp=EARGF(usage())))
 			eprintf("%s: invalid format\n", tmp);
 		fmt = tmp;
 		break;

sort.c

@@ -254,7 +254,7 @@ parse_keydef(struct keydef *kd, char *s, int flags)
 static char *
 skipblank(char *s)
 {
-	while(*s && isblank(*s))
+	while (*s && isblank(*s))
 		s++;
 	return s;
 }
@@ -264,7 +264,7 @@ nextcol(char *s)
 {
 	if (!fieldsep) {
 		s = skipblank(s);
-		while(*s && !isblank(*s))
+		while (*s && !isblank(*s))
 			s++;
 	} else {
 		if (!strchr(s, *fieldsep))

tail.c

@@ -130,7 +130,7 @@ main(int argc, char *argv[])
 			if (fflag && argc == 1) {
 				tmp = NULL;
 				tmpsize = 0;
-				for(;;) {
+				for (;;) {
 					while (getline(&tmp, &tmpsize, fp) != -1) {
 						fputs(tmp, stdout);
 						fflush(stdout);