Hoist common assignments out of a series of if/if-else statements
authorkrw <krw@openbsd.org>
Sat, 2 May 2015 14:29:32 +0000 (14:29 +0000)
committerkrw <krw@openbsd.org>
Sat, 2 May 2015 14:29:32 +0000 (14:29 +0000)
in get_token(). Simplifies code and shrinks future diff.

No intentional functional change.

sbin/dhclient/conflex.c
usr.sbin/dhcpd/conflex.c

index df77fb0..f1e3090 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: conflex.c,v 1.30 2015/05/02 12:37:35 krw Exp $        */
+/*     $OpenBSD: conflex.c,v 1.31 2015/05/02 14:29:32 krw Exp $        */
 
 /* Lexical scanner for dhclient config file. */
 
@@ -147,25 +147,19 @@ get_token(FILE *cfile)
                        skip_to_eol(cfile);
                        continue;
                }
+               lexline = l;
+               lexchar = p;
                if (c == '"') {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_string(cfile);
                        break;
                }
                if ((isascii(c) && isdigit(c)) || c == '-') {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_number(c, cfile);
                        break;
                } else if (isascii(c) && isalpha(c)) {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_num_or_name(c, cfile);
                        break;
                } else {
-                       lexline = l;
-                       lexchar = p;
                        tb[0] = c;
                        tb[1] = 0;
                        tval = tb;
index ef41ae8..5f31921 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: conflex.c,v 1.13 2015/05/02 12:37:35 krw Exp $        */
+/*     $OpenBSD: conflex.c,v 1.14 2015/05/02 14:29:32 krw Exp $        */
 
 /* Lexical scanner for dhcpd config file... */
 
@@ -131,25 +131,19 @@ get_token(FILE *cfile)
                        skip_to_eol(cfile);
                        continue;
                }
+               lexline = l;
+               lexchar = p;
                if (c == '"') {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_string(cfile);
                        break;
                }
                if ((isascii(c) && isdigit(c)) || c == '-') {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_number(c, cfile);
                        break;
                } else if (isascii(c) && isalpha(c)) {
-                       lexline = l;
-                       lexchar = p;
                        ttok = read_num_or_name(c, cfile);
                        break;
                } else {
-                       lexline = l;
-                       lexchar = p;
                        tb[0] = c;
                        tb[1] = 0;
                        tval = tb;