From: Sebastian Harl <sh@tokkee.org>
Date: Mon, 30 Sep 2013 16:24:33 +0000 (+0200)
Subject: utils llist: Added sdb_llist_remove().
X-Git-Tag: sysdb-0.1.0~336^2~66
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=982e6a7ccff9493d3faec0d0327634c2a13ac019;p=sysdb.git

utils llist: Added sdb_llist_remove().
---

diff --git a/src/include/utils/llist.h b/src/include/utils/llist.h
index 298c6ce..af8285f 100644
--- a/src/include/utils/llist.h
+++ b/src/include/utils/llist.h
@@ -146,6 +146,20 @@ sdb_llist_search(sdb_llist_t *list,
 sdb_object_t *
 sdb_llist_search_by_name(sdb_llist_t *list, const char *key);
 
+/*
+ * sdb_llist_remove:
+ * Removes and returns the first matchin element of the list. The ref-count of
+ * the item will not be changed, that is, if the element will not be used any
+ * further, it should be de-referenced by the caller.
+ *
+ * Returns:
+ *  - a pointer to the first matching object
+ *  - NULL else
+ */
+sdb_object_t *
+sdb_llist_remove(sdb_llist_t *list,
+		sdb_llist_lookup_cb lookup, void *user_data);
+
 /*
  * sdb_llist_shift:
  * Removes and returns the first element of the list. The ref-count of the
diff --git a/src/utils/llist.c b/src/utils/llist.c
index 613e53c..3f54786 100644
--- a/src/utils/llist.c
+++ b/src/utils/llist.c
@@ -115,6 +115,21 @@ sdb_llist_insert_after(sdb_llist_t *list, sdb_llist_elem_t *elem,
 	return 0;
 } /* sdb_llist_insert_after */
 
+static sdb_llist_elem_t *
+llist_search(sdb_llist_t *list,
+		sdb_llist_lookup_cb lookup, void *user_data)
+{
+	sdb_llist_elem_t *elem;
+
+	if ((! list) || (! lookup))
+		return NULL;
+
+	for (elem = list->head; elem; elem = elem->next)
+		if (! lookup(elem->obj, user_data))
+			break;
+	return elem;
+} /* llist_search */
+
 static sdb_object_t *
 sdb_llist_remove_elem(sdb_llist_t *list, sdb_llist_elem_t *elem)
 {
@@ -296,16 +311,10 @@ sdb_llist_search(sdb_llist_t *list,
 {
 	sdb_llist_elem_t *elem;
 
-	if ((! list) || (! lookup))
-		return NULL;
-
 	pthread_rwlock_rdlock(&list->lock);
-
-	for (elem = list->head; elem; elem = elem->next)
-		if (! lookup(elem->obj, user_data))
-			break;
-
+	elem = llist_search(list, lookup, user_data);
 	pthread_rwlock_unlock(&list->lock);
+
 	if (elem)
 		return elem->obj;
 	return NULL;
@@ -332,6 +341,22 @@ sdb_llist_search_by_name(sdb_llist_t *list, const char *key)
 	return NULL;
 } /* sdb_llist_search_by_name */
 
+sdb_object_t *
+sdb_llist_remove(sdb_llist_t *list,
+		sdb_llist_lookup_cb lookup, void *user_data)
+{
+	sdb_llist_elem_t *elem;
+	sdb_object_t *obj = NULL;
+
+	pthread_rwlock_wrlock(&list->lock);
+	elem = llist_search(list, lookup, user_data);
+	if (elem)
+		obj = sdb_llist_remove_elem(list, elem);
+	pthread_rwlock_unlock(&list->lock);
+
+	return obj;
+} /* sdb_llist_remove */
+
 sdb_object_t *
 sdb_llist_shift(sdb_llist_t *list)
 {