[src/trunk]: src/usr.bin/make make: reduce scope of the list of wildcard targ...

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

details:   https://anonhg.NetBSD.org/src/rev/546619f18677
branches:  trunk
changeset: 1029251:546619f18677
user:      rillig <rillig%NetBSD.org@localhost>
date:      Tue Dec 28 16:59:09 2021 +0000

description:
make: reduce scope of the list of wildcard target names

The list is only used when a single target name is parsed, in case the
name contains wildcards.  There is no need to keep it any longer or
reuse it.

Clean up outdated and redundant comments.

No functional change.

diffstat:

 usr.bin/make/parse.c |  54 ++++++++++++++-------------------------------------
 1 files changed, 15 insertions(+), 39 deletions(-)

diffs (116 lines):

diff -r 3e089992d644 -r 546619f18677 usr.bin/make/parse.c
--- a/usr.bin/make/parse.c      Tue Dec 28 16:35:43 2021 +0000
+++ b/usr.bin/make/parse.c      Tue Dec 28 16:59:09 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: parse.c,v 1.594 2021/12/28 16:35:43 rillig Exp $       */
+/*     $NetBSD: parse.c,v 1.595 2021/12/28 16:59:09 rillig Exp $       */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -109,7 +109,7 @@
 #include "pathnames.h"
 
 /*     "@(#)parse.c    8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: parse.c,v 1.594 2021/12/28 16:35:43 rillig Exp $");
+MAKE_RCSID("$NetBSD: parse.c,v 1.595 2021/12/28 16:59:09 rillig Exp $");
 
 /* types and constants */
 
@@ -1177,32 +1177,19 @@
 }
 
 static void
-ParseDependencyTargetMundane(char *targetName, StringList *targetNames)
+ParseDependencyTargetMundane(char *targetName)
 {
+       StringList targetNames = LST_INIT;
+
        if (Dir_HasWildcards(targetName)) {
-               /*
-                * Targets are to be sought only in the current directory,
-                * so create an empty path for the thing. Note we need to
-                * use Dir_Destroy in the destruction of the path as the
-                * Dir module could have added a directory to the path...
-                */
                SearchPath *emptyPath = SearchPath_New();
-
-               SearchPath_Expand(emptyPath, targetName, targetNames);
-
+               SearchPath_Expand(emptyPath, targetName, &targetNames);
                SearchPath_Free(emptyPath);
-       } else {
-               /*
-                * No wildcards, but we want to avoid code duplication,
-                * so create a list with the word on it.
-                */
-               Lst_Append(targetNames, targetName);
-       }
-
-       /* Apply the targets. */
-
-       while (!Lst_IsEmpty(targetNames)) {
-               char *targName = Lst_Dequeue(targetNames);
+       } else
+               Lst_Append(&targetNames, targetName);
+
+       while (!Lst_IsEmpty(&targetNames)) {
+               char *targName = Lst_Dequeue(&targetNames);
                GNode *gn = Suff_IsTransform(targName)
                    ? Suff_AddTransform(targName)
                    : Targ_GetNode(targName);
@@ -1395,8 +1382,7 @@
                       const char *lstart,
                       ParseSpecial *inout_special,
                       GNodeType *inout_targetAttr,
-                      SearchPathList **inout_paths,
-                      StringList *targetNames)
+                      SearchPathList **inout_paths)
 {
        char *cp;
        char *tgt = *inout_line;
@@ -1462,7 +1448,7 @@
                 * the end of the targets list
                 */
                if (*inout_special == SP_NOT && *tgt != '\0')
-                       ParseDependencyTargetMundane(tgt, targetNames);
+                       ParseDependencyTargetMundane(tgt);
                else if (*inout_special == SP_PATH && *tgt != '.' &&
                         *tgt != '\0')
                        Parse_Error(PARSE_WARNING, "Extra target (%s) ignored",
@@ -1650,9 +1636,7 @@
        SearchPathList *paths;  /* search paths to alter when parsing a list
                                 * of .PATH targets */
        GNodeType targetAttr;   /* from special sources */
-       /* target names to be found and added to the targets list */
-       StringList targetNames = LST_INIT;
-       char *lstart = line;
+       const char *lstart = line;
 
        /*
         * In special targets, the children are linked as children of the
@@ -1667,16 +1651,9 @@
 
        /* XXX: don't use 'line' as an iterator variable */
        if (!ParseDependencyTargets(&cp, &line, lstart, &special,
-           &targetAttr, &paths, &targetNames))
+           &targetAttr, &paths))
                goto out;
 
-       /*
-        * Don't need the list of target names anymore.
-        * The targets themselves are now in the global variable 'targets'.
-        */
-       Lst_Done(&targetNames);
-       Lst_Init(&targetNames);
-
        if (!Lst_IsEmpty(targets))
                ParseDependencyCheckSpecial(special);
 
@@ -1691,7 +1668,6 @@
 out:
        if (paths != NULL)
                Lst_Free(paths);
-       Lst_Done(&targetNames);
 }
 
 typedef struct VarAssignParsed {