[src/trunk]: src/usr.bin/xlint/lint1 lint: merge duplicate code in init_using...

[rillig <rillig%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/ffe53688075b
branches:  trunk
changeset: 953902:ffe53688075b
user:      rillig <rillig%NetBSD.org@localhost>
date:      Tue Mar 23 20:14:55 2021 +0000

description:
lint: merge duplicate code in init_using_expr

No functional change.

diffstat:

 usr.bin/xlint/lint1/init.c |  33 ++++++++++++---------------------
 1 files changed, 12 insertions(+), 21 deletions(-)

diffs (86 lines):

diff -r 681b576fc744 -r ffe53688075b usr.bin/xlint/lint1/init.c
--- a/usr.bin/xlint/lint1/init.c        Tue Mar 23 19:13:46 2021 +0000
+++ b/usr.bin/xlint/lint1/init.c        Tue Mar 23 20:14:55 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: init.c,v 1.112 2021/03/23 18:51:43 rillig Exp $        */
+/*     $NetBSD: init.c,v 1.113 2021/03/23 20:14:55 rillig Exp $        */
 
 /*
  * Copyright (c) 1994, 1995 Jochen Pohl
@@ -37,7 +37,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: init.c,v 1.112 2021/03/23 18:51:43 rillig Exp $");
+__RCSID("$NetBSD: init.c,v 1.113 2021/03/23 20:14:55 rillig Exp $");
 #endif
 
 #include <stdlib.h>
@@ -980,10 +980,8 @@
        debug_step("expr:");
        debug_node(tn, debug_ind + 1);
 
-       if (initerr || tn == NULL) {
-               debug_leave();
-               return;
-       }
+       if (initerr || tn == NULL)
+               goto done;
 
        sclass = initsym->s_scl;
 
@@ -1007,8 +1005,7 @@
                tn = build(ASSIGN, ln, tn);
                expr(tn, false, false, false, false);
                /* XXX: why not clean up the initstack here already? */
-               debug_leave();
-               return;
+               goto done;
        }
 
        initstack_pop_nobrace();
@@ -1016,17 +1013,12 @@
        if (init_array_using_string(tn)) {
                debug_step("after initializing the string:");
                /* XXX: why not clean up the initstack here already? */
-               debug_initstack();
-               debug_leave();
-               return;
+               goto done_initstack;
        }
 
        initstack_next_nobrace();
-       if (initerr || tn == NULL) {
-               debug_initstack();
-               debug_leave();
-               return;
-       }
+       if (initerr || tn == NULL)
+               goto done_initstack;
 
        initstk->i_remaining--;
        debug_step("%d elements remaining", initstk->i_remaining);
@@ -1048,11 +1040,8 @@
 
        debug_step("typeok '%s', '%s'",
            type_name(ln->tn_type), type_name(tn->tn_type));
-       if (!typeok(INIT, 0, ln, tn)) {
-               debug_initstack();
-               debug_leave();
-               return;
-       }
+       if (!typeok(INIT, 0, ln, tn))
+               goto done_initstack;
 
        /*
         * Store the tree memory. This is necessary because otherwise
@@ -1072,7 +1061,9 @@
 
        check_non_constant_initializer(tn, sclass);
 
+done_initstack:
        debug_initstack();
+done:
        debug_leave();
 }