[src/trunk]: src/usr.bin/make make(1): merge local variables in Compat_Run

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

details:   https://anonhg.NetBSD.org/src/rev/8a443d10ab1c
branches:  trunk
changeset: 946734:8a443d10ab1c
user:      rillig <rillig%NetBSD.org@localhost>
date:      Mon Dec 07 01:10:03 2020 +0000

description:
make(1): merge local variables in Compat_Run

diffstat:

 usr.bin/make/compat.c |  22 ++++++++++------------
 1 files changed, 10 insertions(+), 12 deletions(-)

diffs (71 lines):

diff -r 65ca41b5f1c0 -r 8a443d10ab1c usr.bin/make/compat.c
--- a/usr.bin/make/compat.c     Mon Dec 07 01:07:13 2020 +0000
+++ b/usr.bin/make/compat.c     Mon Dec 07 01:10:03 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: compat.c,v 1.201 2020/12/07 01:07:13 rillig Exp $      */
+/*     $NetBSD: compat.c,v 1.202 2020/12/07 01:10:03 rillig Exp $      */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -96,7 +96,7 @@
 #include "pathnames.h"
 
 /*     "@(#)compat.c   8.2 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: compat.c,v 1.201 2020/12/07 01:07:13 rillig Exp $");
+MAKE_RCSID("$NetBSD: compat.c,v 1.202 2020/12/07 01:10:03 rillig Exp $");
 
 static GNode *curTarg = NULL;
 static pid_t compatChild;
@@ -664,8 +664,7 @@
 Compat_Run(GNodeList *targs)
 {
        GNode *gn = NULL;       /* Current root target */
-       Boolean mainError;
-       Boolean endError;
+       Boolean seenError;
 
        if (!shellName)
                Shell_Init();
@@ -705,7 +704,7 @@
         */
        Make_ExpandUse(targs);
 
-       mainError = FALSE;
+       seenError = FALSE;
        while (!Lst_IsEmpty(targs)) {
                gn = Lst_Dequeue(targs);
                Compat_Make(gn, gn);
@@ -715,7 +714,7 @@
                } else if (gn->made == ABORTED) {
                        printf("`%s' not remade because of errors.\n",
                               gn->name);
-                       mainError = TRUE;
+                       seenError = TRUE;
                }
        }
 
@@ -724,19 +723,18 @@
         * the last one succeeds?  This should not count as overall success.
         */
        if (GNode_IsError(gn))
-               mainError = TRUE;
+               seenError = TRUE;
 
        /*
         * If the user has defined a .END target, run its commands.
         */
-       if (!mainError) {
+       if (!seenError) {
                GNode *endNode = Targ_GetEndNode();
                Compat_Make(endNode, endNode);
-               endError = GNode_IsError(endNode);
-       } else
-               endError = FALSE;
+               seenError = GNode_IsError(endNode);
+       }
 
-       if (mainError || endError) {
+       if (seenError) {
                PrintOnError(gn, "\nStop.");
                exit(1);
        }