Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/make make(1): inline MakeAddChild into ExamineLater



details:   https://anonhg.NetBSD.org/src/rev/24ede83b66ee
branches:  trunk
changeset: 942454:24ede83b66ee
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Nov 08 11:05:58 2020 +0000

description:
make(1): inline MakeAddChild into ExamineLater

This gets rid of a few void pointers and unspecific variable names like
"l" for the list that should have rather been called "examine" all the
time.

Add quotes around placeholders in debug messages.  Especially for targets
like "all" the message had been syntactically misleading.

diffstat:

 usr.bin/make/make.c                         |  40 +++++++++++++++-------------
 usr.bin/make/unit-tests/depsrc-optional.exp |   6 ++--
 2 files changed, 25 insertions(+), 21 deletions(-)

diffs (90 lines):

diff -r a2d03ce6480b -r 24ede83b66ee usr.bin/make/make.c
--- a/usr.bin/make/make.c       Sun Nov 08 10:50:50 2020 +0000
+++ b/usr.bin/make/make.c       Sun Nov 08 11:05:58 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: make.c,v 1.199 2020/11/08 10:50:50 rillig Exp $        */
+/*     $NetBSD: make.c,v 1.200 2020/11/08 11:05:58 rillig Exp $        */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -108,7 +108,7 @@
 #include "job.h"
 
 /*     "@(#)make.c     8.1 (Berkeley) 6/6/93"  */
-MAKE_RCSID("$NetBSD: make.c,v 1.199 2020/11/08 10:50:50 rillig Exp $");
+MAKE_RCSID("$NetBSD: make.c,v 1.200 2020/11/08 11:05:58 rillig Exp $");
 
 /* Sequence # to detect recursion. */
 static unsigned int checked = 1;
@@ -341,21 +341,6 @@
     return oodate;
 }
 
-/* Add the node to the list if it needs to be examined. */
-static int
-MakeAddChild(void *gnp, void *lp)
-{
-    GNode *gn = gnp;
-    GNodeList *l = lp;
-
-    if (!(gn->flags & REMAKE) && !(gn->type & (OP_USE|OP_USEBEFORE))) {
-       DEBUG2(MAKE, "MakeAddChild: need to examine %s%s\n",
-              gn->name, gn->cohort_num);
-       Lst_Enqueue(l, gn);
-    }
-    return 0;
-}
-
 static void
 PretendAllChildrenAreMade(GNode *pgn)
 {
@@ -1101,6 +1086,25 @@
            break;
 }
 
+static void
+ExamineLater(GNodeList *examine, GNodeList *toBeExamined)
+{
+    ListNode *ln;
+
+    for (ln = toBeExamined->first; ln != NULL; ln = ln->next) {
+       GNode *gn = ln->datum;
+
+       if (gn->flags & REMAKE)
+           continue;
+       if (gn->type & (OP_USE | OP_USEBEFORE))
+           continue;
+
+       DEBUG2(MAKE, "ExamineLater: need to examine \"%s%s\"\n",
+              gn->name, gn->cohort_num);
+       Lst_Enqueue(examine, gn);
+    }
+}
+
 /* Expand .USE nodes and create a new targets list.
  *
  * Input:
@@ -1177,7 +1181,7 @@
        }
 
        if (gn->unmade != 0)
-           Lst_ForEachUntil(gn->children, MakeAddChild, examine);
+           ExamineLater(examine, gn->children);
     }
 
     Lst_Free(examine);
diff -r a2d03ce6480b -r 24ede83b66ee usr.bin/make/unit-tests/depsrc-optional.exp
--- a/usr.bin/make/unit-tests/depsrc-optional.exp       Sun Nov 08 10:50:50 2020 +0000
+++ b/usr.bin/make/unit-tests/depsrc-optional.exp       Sun Nov 08 11:05:58 2020 +0000
@@ -1,8 +1,8 @@
 Make_ExpandUse: examine all
-MakeAddChild: need to examine important
+ExamineLater: need to examine "important"
 Make_ExpandUse: examine important
-MakeAddChild: need to examine optional
-MakeAddChild: need to examine optional-cohort
+ExamineLater: need to examine "optional"
+ExamineLater: need to examine "optional-cohort"
 Make_ExpandUse: examine optional
 Make_ExpandUse: examine optional-cohort
 Examining optional...non-existent...up-to-date.



Home | Main Index | Thread Index | Old Index