Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/make make(1): remove unused Lst_ForEach
details: https://anonhg.NetBSD.org/src/rev/8c8dd1f0081e
branches: trunk
changeset: 945070:8c8dd1f0081e
user: rillig <rillig%NetBSD.org@localhost>
date: Mon Oct 19 21:41:31 2020 +0000
description:
make(1): remove unused Lst_ForEach
All of its uses have been inlined since iterating through a linked list
is trivial. This avoids the cumbersome callback functions with void
pointer parameters, allowing the compiler to perform better type checks.
diffstat:
usr.bin/make/lst.c | 12 ++----------
usr.bin/make/lst.h | 6 +-----
2 files changed, 3 insertions(+), 15 deletions(-)
diffs (53 lines):
diff -r d0771b65bbf6 -r 8c8dd1f0081e usr.bin/make/lst.c
--- a/usr.bin/make/lst.c Mon Oct 19 21:38:10 2020 +0000
+++ b/usr.bin/make/lst.c Mon Oct 19 21:41:31 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: lst.c,v 1.77 2020/10/18 13:02:10 rillig Exp $ */
+/* $NetBSD: lst.c,v 1.78 2020/10/19 21:41:31 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -34,7 +34,7 @@
#include "make.h"
-MAKE_RCSID("$NetBSD: lst.c,v 1.77 2020/10/18 13:02:10 rillig Exp $");
+MAKE_RCSID("$NetBSD: lst.c,v 1.78 2020/10/19 21:41:31 rillig Exp $");
/* Allocate and initialize a list node.
*
@@ -308,14 +308,6 @@
return NULL;
}
-void
-Lst_ForEach(List *list, LstActionProc proc, void *procData)
-{
- ListNode *node;
- for (node = list->first; node != NULL; node = node->next)
- proc(node->datum, procData);
-}
-
/* Apply the given function to each element of the given list. The function
* should return 0 if traversal should continue and non-zero if it should
* abort. */
diff -r d0771b65bbf6 -r 8c8dd1f0081e usr.bin/make/lst.h
--- a/usr.bin/make/lst.h Mon Oct 19 21:38:10 2020 +0000
+++ b/usr.bin/make/lst.h Mon Oct 19 21:41:31 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: lst.h,v 1.72 2020/10/18 17:19:54 rillig Exp $ */
+/* $NetBSD: lst.h,v 1.73 2020/10/19 21:41:31 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -185,10 +185,6 @@
/* Iterating over a list, using a callback function */
-/* Apply a function to each datum of the list.
- * The function must not modify the structure of the list, for example by
- * adding or removing nodes. */
-void Lst_ForEach(List *, LstActionProc, void *);
/* Apply a function to each datum of the list, until the callback function
* returns non-zero. */
int Lst_ForEachUntil(List *, LstActionUntilProc, void *);
Home |
Main Index |
Thread Index |
Old Index