From 6b205041c5829dfdf00da8b20afa8f91814778be Mon Sep 17 00:00:00 2001 From: "Rafael G. Martins" Date: Wed, 1 May 2019 23:24:17 +0200 Subject: Revert "common: added bc_slist_remove" This reverts commit 0cfb3dad5de015b658b0917c43fabe420781f39f. --- src/common/utils.c | 19 ------------------- src/common/utils.h | 1 - tests/common/check_utils.c | 35 ----------------------------------- 3 files changed, 55 deletions(-) diff --git a/src/common/utils.c b/src/common/utils.c index 1d4a678..692d1ce 100644 --- a/src/common/utils.c +++ b/src/common/utils.c @@ -74,25 +74,6 @@ bc_slist_prepend(bc_slist_t *l, void *data) } -bc_slist_t* -bc_slist_remove(bc_slist_t *l, bc_slist_t *r, bc_free_func_t free_func) -{ - bc_slist_t *p = NULL; - for (bc_slist_t *tmp = l; tmp != NULL; p = tmp, tmp = tmp->next) { - if (tmp == r) { - if (p == NULL) - l = tmp->next; - else - p->next = tmp->next; - if ((free_func != NULL) && (l->data != NULL)) - free_func(tmp->data); - free(tmp); - return l; - } - } -} - - void bc_slist_free_full(bc_slist_t *l, bc_free_func_t free_func) { diff --git a/src/common/utils.h b/src/common/utils.h index 0e938a4..94c3356 100644 --- a/src/common/utils.h +++ b/src/common/utils.h @@ -31,7 +31,6 @@ typedef struct _bc_slist_t { bc_slist_t* bc_slist_append(bc_slist_t *l, void *data); bc_slist_t* bc_slist_prepend(bc_slist_t *l, void *data); -bc_slist_t* bc_slist_remove(bc_slist_t *l, bc_slist_t *r, bc_free_func_t free_func); void bc_slist_free(bc_slist_t *l); void bc_slist_free_full(bc_slist_t *l, bc_free_func_t free_func); size_t bc_slist_length(bc_slist_t *l); diff --git a/tests/common/check_utils.c b/tests/common/check_utils.c index 3bb4888..cc14f4e 100644 --- a/tests/common/check_utils.c +++ b/tests/common/check_utils.c @@ -53,40 +53,6 @@ test_slist_prepend(void **state) } -static void -test_slist_remove(void **state) -{ - bc_slist_t *l = NULL; - l = bc_slist_append(l, bc_strdup("bola")); - l = bc_slist_append(l, bc_strdup("guda")); - l = bc_slist_append(l, bc_strdup("chunda")); - l = bc_slist_append(l, bc_strdup("pumba")); - l = bc_slist_append(l, bc_strdup("bussunda")); - l = bc_slist_remove(l, l->next->next, free); - assert_non_null(l); - assert_string_equal(l->data, "bola"); - assert_string_equal(l->next->data, "guda"); - assert_string_equal(l->next->next->data, "pumba"); - assert_string_equal(l->next->next->next->data, "bussunda"); - assert_null(l->next->next->next->next); - - l = bc_slist_remove(l, l, free); - assert_non_null(l); - assert_string_equal(l->data, "guda"); - assert_string_equal(l->next->data, "pumba"); - assert_string_equal(l->next->next->data, "bussunda"); - assert_null(l->next->next->next); - - l = bc_slist_remove(l, l->next->next, free); - assert_non_null(l); - assert_string_equal(l->data, "guda"); - assert_string_equal(l->next->data, "pumba"); - assert_null(l->next->next); - - bc_slist_free_full(l, free); -} - - static void test_slist_free(void **state) { @@ -1083,7 +1049,6 @@ main(void) // slist unit_test(test_slist_append), unit_test(test_slist_prepend), - unit_test(test_slist_remove), unit_test(test_slist_free), unit_test(test_slist_length), -- cgit v1.2.3-18-g5258