Module Name: src
Committed By: rillig
Date: Wed Oct 21 06:34:27 UTC 2020
Modified Files:
src/usr.bin/make: suff.c
Log Message:
make(1): clean up SuffFindNormalDepsKnown
To generate a diff of this commit:
cvs rdiff -u -r1.202 -r1.203 src/usr.bin/make/suff.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/usr.bin/make/suff.c
diff -u src/usr.bin/make/suff.c:1.202 src/usr.bin/make/suff.c:1.203
--- src/usr.bin/make/suff.c:1.202 Wed Oct 21 06:30:30 2020
+++ src/usr.bin/make/suff.c Wed Oct 21 06:34:26 2020
@@ -1,4 +1,4 @@
-/* $NetBSD: suff.c,v 1.202 2020/10/21 06:30:30 rillig Exp $ */
+/* $NetBSD: suff.c,v 1.203 2020/10/21 06:34:26 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -129,7 +129,7 @@
#include "dir.h"
/* "@(#)suff.c 8.4 (Berkeley) 3/21/94" */
-MAKE_RCSID("$NetBSD: suff.c,v 1.202 2020/10/21 06:30:30 rillig Exp $");
+MAKE_RCSID("$NetBSD: suff.c,v 1.203 2020/10/21 06:34:26 rillig Exp $");
#define SUFF_DEBUG0(text) DEBUG0(SUFF, text)
#define SUFF_DEBUG1(fmt, arg1) DEBUG1(SUFF, fmt, arg1)
@@ -1593,49 +1593,37 @@ SuffFindNormalDepsKnown(const struct Suf
const char *eoname, const char *sopref,
SrcList *srcs, SrcList *targs)
{
- SuffListNode *ln = sufflist->first;
-
- while (ln != NULL) {
- /*
- * Look for next possible suffix...
- */
- ln = Lst_FindFrom(sufflist, ln, SuffSuffIsSuffix, sd);
+ SuffListNode *ln;
+ const char *eopref;
+ Src *targ;
- if (ln != NULL) {
- const char *eopref;
+ for (ln = sufflist->first; ln != NULL; ln = ln->next) {
+ if (!SuffSuffIsSuffix(ln->datum, sd))
+ continue;
- /*
- * Allocate a Src structure to which things can be transformed
- */
- Src *targ = bmake_malloc(sizeof(Src));
- targ->file = bmake_strdup(gn->name);
- targ->suff = ln->datum;
- targ->suff->refCount++;
- targ->node = gn;
- targ->parent = NULL;
- targ->children = 0;
+ targ = bmake_malloc(sizeof(Src));
+ targ->file = bmake_strdup(gn->name);
+ targ->suff = ln->datum;
+ targ->suff->refCount++;
+ targ->node = gn;
+ targ->parent = NULL;
+ targ->children = 0;
#ifdef DEBUG_SRC
- targ->cp = Lst_New();
+ targ->cp = Lst_New();
#endif
- eopref = eoname - targ->suff->nameLen;
- targ->pref = bmake_strsedup(sopref, eopref);
-
- /*
- * Add nodes from which the target can be made
- */
- SuffAddLevel(srcs, targ);
+ eopref = eoname - targ->suff->nameLen;
+ targ->pref = bmake_strsedup(sopref, eopref);
- /*
- * Record the target so we can nuke it
- */
- Lst_Append(targs, targ);
+ /*
+ * Add nodes from which the target can be made
+ */
+ SuffAddLevel(srcs, targ);
- /*
- * Search from this suffix's successor...
- */
- ln = ln->next;
- }
+ /*
+ * Record the target so we can nuke it
+ */
+ Lst_Append(targs, targ);
}
}