o Remove a useless list where just all command lines are stuffed
authorMax Okumoto <okumoto@dragonflybsd.org>
Fri, 17 Dec 2004 08:09:58 +0000 (08:09 +0000)
committerMax Okumoto <okumoto@dragonflybsd.org>
Fri, 17 Dec 2004 08:09:58 +0000 (08:09 +0000)
  onto, never used and just freed at the end. The idea might have
  been to be able to free all the strings, but what's the point to
  free just before exiting?

o Remove a list that used to hold all the GNodes just to be able
  to free them just before exiting.

o Remove a list that was just used to free all variables just before
  exiting.

Taken-from: FreeBSD
Author: harti

usr.bin/make/parse.c
usr.bin/make/targ.c
usr.bin/make/var.c

index 33fedf5..9d41212 100644 (file)
@@ -37,7 +37,7 @@
  *
  * @(#)parse.c 8.3 (Berkeley) 3/19/94
  * $FreeBSD: src/usr.bin/make/parse.c,v 1.22.2.2 2004/07/10 08:14:42 eik Exp $
- * $DragonFly: src/usr.bin/make/parse.c,v 1.25 2004/12/17 00:02:57 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/parse.c,v 1.26 2004/12/17 08:09:58 okumoto Exp $
  */
 
 /*-
 #define        CONTINUE        1
 #define        DONE            0
 static Lst          *targets;  /* targets we're working on */
-static Lst         *targCmds;  /* command lines for targets */
 static Boolean     inLine;     /* true if currently in a dependency
                                 * line or its commands */
 static int         fatals = 0;
@@ -2451,7 +2450,6 @@ Parse_File(char *name, FILE *stream)
                         * commands of all targets in the dependency spec
                         */
                        Lst_ForEach(targets, ParseAddCmd, cp);
-                       Lst_AtEnd(targCmds, line);
                        continue;
                    } else {
                        Parse_Error(PARSE_FATAL,
@@ -2549,14 +2547,12 @@ Parse_Init(void)
     parseIncPath = Lst_Init();
     sysIncPath = Lst_Init();
     includes = Lst_Init();
-    targCmds = Lst_Init();
 }
 
 void
 Parse_End(void)
 {
 
-    Lst_Destroy(targCmds, free);
     if (targets)
        Lst_Destroy(targets, NOFREE);
     Lst_Destroy(sysIncPath, Dir_Destroy);
index 3cc7ae7..62bced8 100644 (file)
@@ -37,7 +37,7 @@
  *
  * @(#)targ.c  8.2 (Berkeley) 3/19/94
  * $FreeBSD: src/usr.bin/make/targ.c,v 1.10 1999/09/11 13:08:02 hoek Exp $
- * $DragonFly: src/usr.bin/make/targ.c,v 1.17 2004/12/17 07:56:08 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/targ.c,v 1.18 2004/12/17 08:09:58 okumoto Exp $
  */
 
 /*-
@@ -88,7 +88,6 @@
 #include         "dir.h"
 
 static Lst        *allTargets; /* the list of all targets found so far */
-static Lst       *allGNs;      /* List of all the GNodes */
 static Hash_Table targets;     /* a hash table of same */
 
 #define        HTSIZE  191             /* initial size of hash table */
@@ -96,7 +95,6 @@ static Hash_Table targets;    /* a hash table of same */
 static int TargPrintOnlySrc(void *, void *);
 static int TargPrintName(void *, void *);
 static int TargPrintNode(void *, void *);
-static void TargFreeGN(void *);
 
 /*-
  *-----------------------------------------------------------------------
@@ -135,8 +133,6 @@ Targ_End(void)
 {
 
     Lst_Destroy(allTargets, NOFREE);
-    if (allGNs)
-       Lst_Destroy(allGNs, TargFreeGN);
     Hash_DeleteTable(&targets);
 }
 
@@ -182,44 +178,9 @@ Targ_NewGN(char *name)
     gn->commands = Lst_Init();
     gn->suffix = NULL;
 
-    if (allGNs == NULL)
-       allGNs = Lst_Init();
-    Lst_AtEnd(allGNs, (void *)gn);
-
     return (gn);
 }
 
-/*-
- *-----------------------------------------------------------------------
- * TargFreeGN  --
- *     Destroy a GNode
- *
- * Results:
- *     None.
- *
- * Side Effects:
- *     None.
- *-----------------------------------------------------------------------
- */
-static void
-TargFreeGN(void *gnp)
-{
-    GNode *gn = gnp;
-
-    free(gn->name);
-    free(gn->path);
-
-    Lst_Destroy(gn->iParents, NOFREE);
-    Lst_Destroy(gn->cohorts, NOFREE);
-    Lst_Destroy(gn->parents, NOFREE);
-    Lst_Destroy(gn->children, NOFREE);
-    Lst_Destroy(gn->successors, NOFREE);
-    Lst_Destroy(gn->preds, NOFREE);
-    Lst_Destroy(gn->context, NOFREE);
-    Lst_Destroy(gn->commands, NOFREE);
-    free(gn);
-}
-
 /*-
  *-----------------------------------------------------------------------
  * Targ_FindNode  --
index 4fe1350..6810889 100644 (file)
@@ -37,7 +37,7 @@
  *
  * @(#)var.c   8.3 (Berkeley) 3/19/94
  * $FreeBSD: src/usr.bin/make/var.c,v 1.16.2.3 2002/02/27 14:18:57 cjc Exp $
- * $DragonFly: src/usr.bin/make/var.c,v 1.25 2004/12/17 07:53:57 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/var.c,v 1.26 2004/12/17 08:09:58 okumoto Exp $
  */
 
 /*-
@@ -120,14 +120,12 @@ static char       varNoError[] = "";
  * The four contexts are searched in the reverse order from which they are
  * listed.
  */
-GNode          *VAR_GLOBAL;   /* variables from the makefile */
-GNode          *VAR_CMD;      /* variables defined on the command-line */
+GNode  *VAR_GLOBAL;    /* variables from the makefile */
+GNode  *VAR_CMD;       /* variables defined on the command-line */
 
-static Lst     *allVars;      /* List of all variables */
-
-#define FIND_CMD       0x1   /* look in VAR_CMD when searching */
-#define FIND_GLOBAL    0x2   /* look in VAR_GLOBAL as well */
-#define FIND_ENV       0x4   /* look in the environment also */
+#define        FIND_CMD        0x1     /* look in VAR_CMD when searching */
+#define        FIND_GLOBAL     0x2     /* look in VAR_GLOBAL as well */
+#define        FIND_ENV        0x4     /* look in the environment also */
 
 static void VarPossiblyExpand(char **, GNode *);
 static Var *VarFind(char *, GNode *, int);
@@ -334,7 +332,6 @@ VarAdd(char *name, char *val, GNode *ctxt)
     v->flags = 0;
 
     Lst_AtFront(ctxt->context, v);
-    Lst_AtEnd(allVars, v);
     DEBUGF(VAR, ("%s:%s = %s\n", ctxt->name, name, val));
 }
 
@@ -381,13 +378,8 @@ Var_Delete(char *name, GNode *ctxt)
     DEBUGF(VAR, ("%s:delete %s\n", ctxt->name, name));
     ln = Lst_Find(ctxt->context, name, VarCmp);
     if (ln != NULL) {
-       Var       *v;
-
-       v = Lst_Datum(ln);
+       VarDelete(Lst_Datum(ln));
        Lst_Remove(ctxt->context, ln);
-       ln = Lst_Member(allVars, v);
-       Lst_Remove(allVars, ln);
-       VarDelete(v);
     }
 }
 
@@ -1946,15 +1938,12 @@ Var_Init(void)
 
     VAR_GLOBAL = Targ_NewGN("Global");
     VAR_CMD = Targ_NewGN("Command");
-    allVars = Lst_Init();
 
 }
 
 void
 Var_End(void)
 {
-
-    Lst_Destroy(allVars, VarDelete);
 }