From 38b205d0a4d7c29f7ba32f3f96e4ded9790d0af2 Mon Sep 17 00:00:00 2001
From: Michael Brown
Date: Fri, 14 Oct 2011 13:35:05 +0100
Subject: [list] Tidy up naming convention for list_contains() and friends

Signed-off-by: Michael Brown <mcb30@ipxe.org>
---
 src/include/ipxe/list.h | 56 ++++++++++++++++++++++++++++++++++++++++---------
 1 file changed, 46 insertions(+), 10 deletions(-)

(limited to 'src/include')

diff --git a/src/include/ipxe/list.h b/src/include/ipxe/list.h
index 4905aaa3a..8dc74139d 100644
--- a/src/include/ipxe/list.h
+++ b/src/include/ipxe/list.h
@@ -170,6 +170,18 @@ static inline int list_empty ( const struct list_head *list ) {
 	  ( type * ) NULL :				\
 	  list_entry ( (list)->next, type, member ) )
 
+/**
+ * Iterate over a list
+ *
+ * @v pos		Iterator
+ * @v head		List head
+ */
+#define list_for_each( pos, head )					      \
+	for ( list_check ( (head) ),					      \
+	      pos = (head)->next;					      \
+	      pos != (head);						      \
+	      pos = (pos)->next )
+
 /**
  * Iterate over entries in a list
  *
@@ -212,6 +224,38 @@ static inline int list_empty ( const struct list_head *list ) {
 	      pos = tmp,						      \
 	      tmp = list_entry ( tmp->member.next, typeof ( *tmp ), member ) )
 
+/**
+ * Test if list contains a specified entry
+ *
+ * @v entry		Entry
+ * @v head		List head
+ * @ret present		List contains specified entry
+ */
+static inline int list_contains ( struct list_head *entry,
+				  struct list_head *head ) {
+	struct list_head *tmp;
+
+	list_for_each ( tmp, head ) {
+		if ( tmp == entry )
+			return 1;
+	}
+	return 0;
+}
+#define list_contains( entry, head ) ( {		\
+	list_check ( (head) );				\
+	list_check ( (entry) );				\
+	list_contains ( (entry), (head) ); } )
+
+/**
+ * Test if list contains a specified entry
+ *
+ * @v entry		Entry
+ * @v head		List head
+ * @ret present		List contains specified entry
+ */
+#define list_contains_entry( entry, head, member )	\
+	list_contains ( &(entry)->member, (head) )
+
 /**
  * Check list contains a specified entry
  *
@@ -219,16 +263,8 @@ static inline int list_empty ( const struct list_head *list ) {
  * @v head		List head
  * @v member		Name of list field within iterator's type
  */
-#define list_check_contains( entry, head, member ) do {		\
-	if ( ASSERTING ) {					\
-		typeof ( entry ) tmp;				\
-		int found = 0;					\
-		list_for_each_entry ( tmp, head, member ) {	\
-			if ( tmp == entry )			\
-				found = 1;			\
-		}						\
-		assert ( found );				\
-	}							\
+#define list_check_contains_entry( entry, head, member ) do {		      \
+	assert ( list_contains_entry ( (entry), (head), member ) );	      \
 	} while ( 0 )
 
 #endif /* _IPXE_LIST_H */
-- 
cgit v1.2.3-55-g7522