set: drop unused set_make() function (#8879)

The function causes compiler error when built with '-Ddebug=hashmap',
and is not used anymore. Let's drop it.

diff --git a/src/basic/meson.build b/src/basic/meson.build
index 4f63fcef1ce557a2c83618a81a22531d9e184266..cff1b9d18ff46b6e80af84a33567f8e8475ea4a9 100644 (file)
--- a/src/basic/meson.build
+++ b/src/basic/meson.build
@@ -163,7 +163,6 @@ basic_sources = files('''
         securebits.h
         selinux-util.c
         selinux-util.h
-        set.c
         set.h
         sigbus.c
         sigbus.h

diff --git a/src/basic/set.c b/src/basic/set.c
deleted file mode 100644 (file)
index 4a10c74..0000000
--- a/src/basic/set.c
+++ /dev/null
@@ -1,49 +0,0 @@
-/* SPDX-License-Identifier: LGPL-2.1+ */
-/***
-  This file is part of systemd.
-
-  Copyright 2017 Lennart Poettering
-***/
-
-#include "alloc-util.h"
-#include "set.h"
-
-int set_make(Set **ret, const struct hash_ops *hash_ops HASHMAP_DEBUG_PARAMS, void *add, ...) {
-        _cleanup_set_free_ Set *s = NULL;
-        int r;
-
-        assert(ret);
-
-        s = set_new(hash_ops HASHMAP_DEBUG_PASS_ARGS);
-        if (!s)
-                return -ENOMEM;
-
-        if (add) {
-                va_list ap;
-
-                r = set_put(s, add);
-                if (r < 0)
-                        return r;
-
-                va_start(ap, add);
-
-                for (;;) {
-                        void *arg = va_arg(ap, void*);
-
-                        if (!arg)
-                                break;
-
-                        r = set_put(s, arg);
-                        if (r < 0) {
-                                va_end(ap);
-                                return r;
-                        }
-                }
-
-                va_end(ap);
-        }
-
-        *ret = TAKE_PTR(s);
-
-        return 0;
-}

diff --git a/src/basic/set.h b/src/basic/set.h
index 6f937af9292e0724c2077c02f7749a59a2e58a39..dc0f1e17e62c58db26b6d54669bd65743ef6a919 100644 (file)
--- a/src/basic/set.h
+++ b/src/basic/set.h
@@ -136,5 +136,3 @@ DEFINE_TRIVIAL_CLEANUP_FUNC(Set*, set_free_free);
 
 #define _cleanup_set_free_ _cleanup_(set_freep)
 #define _cleanup_set_free_free_ _cleanup_(set_free_freep)
-
-int set_make(Set **ret, const struct hash_ops *hash_ops HASHMAP_DEBUG_PARAMS, void *add, ...);

diff --git a/src/test/test-set.c b/src/test/test-set.c
index 11884de1f78bdd771887c7c3f55bc980354e9174..9774fbb3801d3bbd90fa3de3f672272214aec325 100644 (file)
--- a/src/test/test-set.c
+++ b/src/test/test-set.c
@@ -66,55 +66,10 @@ static void test_set_put(void) {
         assert_se(set_put(m, (void*) "22") == 0);
 }
 
-static void test_set_make(void) {
-        _cleanup_set_free_ Set *s = NULL;
-
-        assert_se(set_make(&s, NULL, UINT_TO_PTR(4), UINT_TO_PTR(6), UINT_TO_PTR(8), NULL) == 0);
-        assert_se(set_size(s) == 3);
-        assert_se(!set_contains(s, UINT_TO_PTR(0)));
-        assert_se(!set_contains(s, UINT_TO_PTR(1)));
-        assert_se(!set_contains(s, UINT_TO_PTR(2)));
-        assert_se(!set_contains(s, UINT_TO_PTR(3)));
-        assert_se(set_contains(s, UINT_TO_PTR(4)));
-        assert_se(!set_contains(s, UINT_TO_PTR(5)));
-        assert_se(set_contains(s, UINT_TO_PTR(6)));
-        assert_se(!set_contains(s, UINT_TO_PTR(7)));
-        assert_se(set_contains(s, UINT_TO_PTR(8)));
-        assert_se(!set_contains(s, UINT_TO_PTR(9)));
-        s = set_free(s);
-
-        assert_se(set_make(&s, NULL, NULL) == 0);
-        assert_se(set_size(s) == 0);
-        assert_se(!set_contains(s, UINT_TO_PTR(0)));
-        assert_se(!set_contains(s, UINT_TO_PTR(4)));
-        assert_se(!set_contains(s, UINT_TO_PTR(6)));
-        assert_se(!set_contains(s, UINT_TO_PTR(8)));
-        s = set_free(s);
-
-        assert_se(set_make(&s, NULL, UINT_TO_PTR(3), NULL) == 0);
-        assert_se(set_size(s) == 1);
-        assert_se(!set_contains(s, UINT_TO_PTR(0)));
-        assert_se(!set_contains(s, UINT_TO_PTR(1)));
-        assert_se(!set_contains(s, UINT_TO_PTR(2)));
-        assert_se(set_contains(s, UINT_TO_PTR(3)));
-        assert_se(!set_contains(s, UINT_TO_PTR(4)));
-
-        assert_se(set_make(&s, NULL, UINT_TO_PTR(2), UINT_TO_PTR(5), NULL) == 0);
-        assert_se(set_size(s) == 2);
-        assert_se(!set_contains(s, UINT_TO_PTR(0)));
-        assert_se(!set_contains(s, UINT_TO_PTR(1)));
-        assert_se(set_contains(s, UINT_TO_PTR(2)));
-        assert_se(!set_contains(s, UINT_TO_PTR(3)));
-        assert_se(!set_contains(s, UINT_TO_PTR(4)));
-        assert_se(set_contains(s, UINT_TO_PTR(5)));
-        assert_se(!set_contains(s, UINT_TO_PTR(6)));
-}
-
 int main(int argc, const char *argv[]) {
         test_set_steal_first();
         test_set_free_with_destructor();
         test_set_put();
-        test_set_make();
 
         return 0;
 }