[src/trunk]: src/usr.bin/make make(1): reduce memory allocation in meta_oodate

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

details:   https://anonhg.NetBSD.org/src/rev/7dc95fd1e39c
branches:  trunk
changeset: 957487:7dc95fd1e39c
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Nov 29 21:31:55 2020 +0000

description:
make(1): reduce memory allocation in meta_oodate

diffstat:

 usr.bin/make/meta.c |  20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diffs (81 lines):

diff -r 2119354a999b -r 7dc95fd1e39c usr.bin/make/meta.c
--- a/usr.bin/make/meta.c       Sun Nov 29 21:28:06 2020 +0000
+++ b/usr.bin/make/meta.c       Sun Nov 29 21:31:55 2020 +0000
@@ -1,4 +1,4 @@
-/*      $NetBSD: meta.c,v 1.155 2020/11/29 21:28:06 rillig Exp $ */
+/*      $NetBSD: meta.c,v 1.156 2020/11/29 21:31:55 rillig Exp $ */
 
 /*
  * Implement 'meta' mode.
@@ -1090,7 +1090,7 @@
     static size_t tmplen = 0;
     FILE *fp;
     Boolean needOODATE = FALSE;
-    StringList *missingFiles;
+    StringList missingFiles;
     Boolean have_filemon = FALSE;
     void *objdir_freeIt;
 
@@ -1105,7 +1105,7 @@
        goto oodate_out;
     dname = fname3;
 
-    missingFiles = Lst_New();
+    Lst_Init(&missingFiles);
 
     /*
      * We need to check if the target is out-of-date. This includes
@@ -1319,12 +1319,12 @@
                case 'D':               /* unlink */
                    if (*p == '/') {
                        /* remove any missingFiles entries that match p */
-                       StringListNode *ln = missingFiles->first;
+                       StringListNode *ln = missingFiles.first;
                        while (ln != NULL) {
                            StringListNode *next = ln->next;
                            if (path_starts_with(ln->datum, p)) {
                                free(ln->datum);
-                               Lst_Remove(missingFiles, ln);
+                               Lst_Remove(&missingFiles, ln);
                            }
                            ln = next;
                        }
@@ -1390,7 +1390,7 @@
                    if ((link_src != NULL && cached_lstat(p, &cst) < 0) ||
                        (link_src == NULL && cached_stat(p, &cst) < 0)) {
                        if (!meta_ignore(gn, p))
-                           append_if_new(missingFiles, p);
+                           append_if_new(&missingFiles, p);
                    }
                    break;
                check_link_src:
@@ -1473,7 +1473,7 @@
                             * A referenced file outside of CWD is missing.
                             * We cannot catch every eventuality here...
                             */
-                           append_if_new(missingFiles, p);
+                           append_if_new(&missingFiles, p);
                        }
                    }
                    if (buf[0] == 'E') {
@@ -1571,9 +1571,9 @@
        }
 
        fclose(fp);
-       if (!Lst_IsEmpty(missingFiles)) {
+       if (!Lst_IsEmpty(&missingFiles)) {
            DEBUG2(META, "%s: missing files: %s...\n",
-                  fname, (char *)missingFiles->first->datum);
+                  fname, (char *)missingFiles.first->datum);
            oodate = TRUE;
        }
        if (!oodate && !have_filemon && filemonMissing) {
@@ -1598,7 +1598,7 @@
        }
     }
 
-    Lst_Destroy(missingFiles, free);
+    Lst_DoneCall(&missingFiles, free);
 
     if (oodate && needOODATE) {
        /*