]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
basic/env-util: drop now-unused strv_env_set()
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 15 Feb 2021 13:55:40 +0000 (14:55 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Tue, 16 Feb 2021 15:10:14 +0000 (16:10 +0100)
src/basic/env-util.c
src/basic/env-util.h
src/test/test-env-util.c

index 5f2b29d7b5b351350bdc4f998a498fbd6e425ebd..b1313dbfff8d3c09be14fa741658f669366c9d5e 100644 (file)
@@ -434,34 +434,6 @@ int strv_env_assign(char ***l, const char *key, const char *value) {
         return r;
 }
 
-char **strv_env_set(char **x, const char *p) {
-        _cleanup_strv_free_ char **ret = NULL;
-        size_t n, m;
-        char **k;
-
-        /* Overrides the env var setting of p, returns a new copy */
-
-        n = strv_length(x);
-        m = n + 2;
-        if (m < n) /* overflow? */
-                return NULL;
-
-        ret = new(char*, m);
-        if (!ret)
-                return NULL;
-
-        *ret = NULL;
-        k = ret;
-
-        if (env_append(ret, &k, x) < 0)
-                return NULL;
-
-        if (env_append(ret, &k, STRV_MAKE(p)) < 0)
-                return NULL;
-
-        return TAKE_PTR(ret);
-}
-
 char *strv_env_get_n(char **l, const char *name, size_t k, unsigned flags) {
         char **i;
 
index 3f8995bf3db8b99adc2bdf6ceb176b0f78099f10..c7b72b4b471f6fecea6ec0fd8e5f2dafd46b9972 100644 (file)
@@ -42,7 +42,6 @@ bool strv_env_name_or_assignment_is_valid(char **l);
 char **strv_env_merge(size_t n_lists, ...);
 char **strv_env_delete(char **x, size_t n_lists, ...); /* New copy */
 
-char **strv_env_set(char **x, const char *p); /* New copy ... */
 char **strv_env_unset(char **l, const char *p); /* In place ... */
 char **strv_env_unset_many(char **l, ...) _sentinel_;
 int strv_env_replace(char ***l, char *p); /* In place ... */
index ce215fdd36458ec254178b4b257a9314dd0403c2..39be9532b356baf8b14f248ad521a8f911f5b2b2 100644 (file)
@@ -59,24 +59,6 @@ static void test_strv_env_unset(void) {
         assert_se(strv_length(l) == 2);
 }
 
-static void test_strv_env_set(void) {
-        log_info("/* %s */", __func__);
-
-        _cleanup_strv_free_ char **l = NULL, **r = NULL;
-
-        l = strv_new("PIEP", "SCHLUMPF=SMURFF", "NANANANA=YES");
-        assert_se(l);
-
-        r = strv_env_set(l, "WALDO=WALDO");
-        assert_se(r);
-
-        assert_se(streq(r[0], "PIEP"));
-        assert_se(streq(r[1], "SCHLUMPF=SMURFF"));
-        assert_se(streq(r[2], "NANANANA=YES"));
-        assert_se(streq(r[3], "WALDO=WALDO"));
-        assert_se(strv_length(r) == 4);
-}
-
 static void test_strv_env_merge(void) {
         log_info("/* %s */", __func__);
 
@@ -378,7 +360,6 @@ int main(int argc, char *argv[]) {
         test_strv_env_delete();
         test_strv_env_get();
         test_strv_env_unset();
-        test_strv_env_set();
         test_strv_env_merge();
         test_env_strv_get_n();
         test_replace_env(false);