From: espie <espie@openbsd.org>
Date: Sat, 6 Mar 2021 08:31:42 +0000 (+0000)
Subject: put back Lst_Destroy(paths) where it belongs
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=cf6dba2a7e026b3bf62d478aa0f76fc0105c1abc;p=openbsd

put back Lst_Destroy(paths) where it belongs
noticed by jsg@
---

diff --git a/usr.bin/make/parse.c b/usr.bin/make/parse.c
index 35ab1671060..1b29df2300b 100644
--- a/usr.bin/make/parse.c
+++ b/usr.bin/make/parse.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: parse.c,v 1.133 2020/04/20 08:17:33 espie Exp $	*/
+/*	$OpenBSD: parse.c,v 1.134 2021/03/06 08:31:42 espie Exp $	*/
 /*	$NetBSD: parse.c,v 1.29 1997/03/10 21:20:04 christos Exp $	*/
 
 /*
@@ -925,7 +925,6 @@ ParseDoDependency(const char *line)	/* the line to parse */
 			    	ln = Lst_Adv(ln))
 				    Dir_AddDiri(Lst_Datum(ln), line, cp);
 			    break;
-			    Lst_Destroy(&paths, NOFREE);
 			    }
 		    default:
 			    break;
@@ -989,6 +988,11 @@ ParseDoDependency(const char *line)	/* the line to parse */
 		}
 	}
 
+	/* If we're not in the SPEC_PATH case the list is empty, but who 
+	 * cares ?  This is cheap. 
+	 */
+	Lst_Destroy(&paths, NOFREE);
+
 	if (mainNode == NULL) {
 		/* If we have yet to decide on a main target to make, in the
 		 * absence of any user input, we want the first target on