diff --git a/usr.bin/make/arch.c b/usr.bin/make/arch.c index 148be55476e4..6cc89b74de69 100644 --- a/usr.bin/make/arch.c +++ b/usr.bin/make/arch.c @@ -413,28 +413,6 @@ Arch_ParseArchive(char **linePtr, Lst *nodeLst, GNode *ctxt) return (SUCCESS); } -/*- - *----------------------------------------------------------------------- - * ArchFindArchive -- - * See if the given archive is the one we are looking for. Called - * From ArchStatMember and ArchFindMember via Lst_Find with the - * current list element and the name we want. - * - * Results: - * 0 if it is, non-zero if it isn't. - * - * Side Effects: - * None. - * - *----------------------------------------------------------------------- - */ -static int -ArchFindArchive(const void *ar, const void *archName) -{ - - return (strcmp(archName, ((const Arch *)ar)->name)); -} - /*- *----------------------------------------------------------------------- * ArchFindMember -- @@ -723,7 +701,10 @@ ArchStatMember(const char *archive, const char *member, Boolean hash) if (cp != NULL && strcmp(member, RANLIBMAG) != 0) member = cp + 1; - ln = Lst_Find(&archives, archive, ArchFindArchive); + LST_FOREACH(ln, &archives) { + if (strcmp(archive, ((const Arch *)Lst_Datum(ln))->name) == 0) + break; + } if (ln != NULL) { char copy[AR_MAX_NAME_LEN + 1]; diff --git a/usr.bin/make/var.c b/usr.bin/make/var.c index d337502d814e..6bfdceee8079 100644 --- a/usr.bin/make/var.c +++ b/usr.bin/make/var.c @@ -196,24 +196,18 @@ VarDestroy(Var *v, Boolean f) free(v); } -/*- - *----------------------------------------------------------------------- - * VarCmp -- - * See if the given variable matches the named one. Called from - * Lst_Find when searching for a variable of a given name. - * - * Results: - * 0 if they match. non-zero otherwise. - * - * Side Effects: - * none - *----------------------------------------------------------------------- +/* + * Find a variable in a variable list. */ -static int -VarCmp(const void *v, const void *name) +static Var * +VarLookup(Lst *vlist, const char *name) { + LstNode *ln; - return (strcmp(name, ((const Var *)v)->name)); + LST_FOREACH(ln, vlist) + if (strcmp(((const Var *)Lst_Datum(ln))->name, name) == 0) + return (Lst_Datum(ln)); + return (NULL); } /*- @@ -261,7 +255,8 @@ static Var * VarFind(const char *name, GNode *ctxt, int flags) { Boolean localCheckEnvFirst; - LstNode *var; + LstNode *ln; + Var *var; char *env; /* @@ -307,10 +302,12 @@ VarFind(const char *name, GNode *ctxt, int flags) * Note whether this is one of the specific variables we were told * through the -E flag to use environment-variable-override for. */ - if (Lst_Find(&envFirstVars, name, (CompareProc *)strcmp) != NULL) { - localCheckEnvFirst = TRUE; - } else { - localCheckEnvFirst = FALSE; + localCheckEnvFirst = FALSE; + LST_FOREACH(ln, &envFirstVars) { + if (strcmp(Lst_Datum(ln), name) == 0) { + localCheckEnvFirst = TRUE; + break; + } } /* @@ -318,25 +315,25 @@ VarFind(const char *name, GNode *ctxt, int flags) * look for it in VAR_CMD, VAR_GLOBAL and the environment, * in that order, depending on the FIND_* flags in 'flags' */ - var = Lst_Find(&ctxt->context, name, VarCmp); + var = VarLookup(&ctxt->context, name); if (var != NULL) { /* got it */ - return (Lst_Datum(var)); + return (var); } /* not there - try command line context */ if ((flags & FIND_CMD) && (ctxt != VAR_CMD)) { - var = Lst_Find(&VAR_CMD->context, name, VarCmp); + var = VarLookup(&VAR_CMD->context, name); if (var != NULL) - return (Lst_Datum(var)); + return (var); } /* not there - try global context, but only if not -e/-E */ if ((flags & FIND_GLOBAL) && (ctxt != VAR_GLOBAL) && !checkEnvFirst && !localCheckEnvFirst) { - var = Lst_Find(&VAR_GLOBAL->context, name, VarCmp); + var = VarLookup(&VAR_GLOBAL->context, name); if (var != NULL) - return (Lst_Datum(var)); + return (var); } if (!(flags & FIND_ENV)) @@ -352,9 +349,9 @@ VarFind(const char *name, GNode *ctxt, int flags) /* deferred check for the environment (in case of -e/-E) */ if ((checkEnvFirst || localCheckEnvFirst) && (flags & FIND_GLOBAL) && (ctxt != VAR_GLOBAL)) { - var = Lst_Find(&VAR_GLOBAL->context, name, VarCmp); + var = VarLookup(&VAR_GLOBAL->context, name); if (var != NULL) - return (Lst_Datum(var)); + return (var); } return (NULL); } @@ -400,10 +397,12 @@ Var_Delete(const char *name, GNode *ctxt) LstNode *ln; DEBUGF(VAR, ("%s:delete %s\n", ctxt->name, name)); - ln = Lst_Find(&ctxt->context, name, VarCmp); - if (ln != NULL) { - VarDestroy(Lst_Datum(ln), TRUE); - Lst_Remove(&ctxt->context, ln); + LST_FOREACH(ln, &ctxt->context) { + if (strcmp(((const Var *)Lst_Datum(ln))->name, name) == 0) { + VarDestroy(Lst_Datum(ln), TRUE); + Lst_Remove(&ctxt->context, ln); + break; + } } }