strv: add strv_print

Clearer, and spares the temp variable.

diff --git a/src/locale/localectl.c b/src/locale/localectl.c
index 290edcc103bbdbc2866f8972ad982ebc8448d4e5..8c3c8e3f0fd99b499f8e76c5bb97b69130f4c756 100644 (file)
--- a/src/locale/localectl.c
+++ b/src/locale/localectl.c
@@ -432,7 +432,6 @@ static int add_locales_from_libdir (Set *locales) {
 static int list_locales(DBusConnection *bus, char **args, unsigned n) {
         _cleanup_set_free_ Set *locales;
         _cleanup_strv_free_ char **l = NULL;
-        char **j;
         int r;
 
         locales = set_new(string_hash_func, string_compare_func);
@@ -455,8 +454,7 @@ static int list_locales(DBusConnection *bus, char **args, unsigned n) {
 
         pager_open_if_enabled();
 
-        STRV_FOREACH(j, l)
-                puts(*j);
+        strv_print(l);
 
         return 0;
 }
@@ -539,7 +537,6 @@ static int nftw_cb(
 
 static int list_vconsole_keymaps(DBusConnection *bus, char **args, unsigned n) {
         char _cleanup_strv_free_ **l = NULL;
-        char **i;
 
         keymaps = set_new(string_hash_func, string_compare_func);
         if (!keymaps)
@@ -566,9 +563,7 @@ static int list_vconsole_keymaps(DBusConnection *bus, char **args, unsigned n) {
 
         pager_open_if_enabled();
 
-        STRV_FOREACH(i, l)
-                puts(*i);
-
+        strv_print(l);
 
         return 0;
 }

diff --git a/src/shared/strv.c b/src/shared/strv.c
index 2d556f4a076a664a71ffc70f0bd77d675e531bde..fc6104ffeaaba438065c3a9e594f923a429e21b1 100644 (file)
--- a/src/shared/strv.c
+++ b/src/shared/strv.c
@@ -774,3 +774,13 @@ char **strv_sort(char **l) {
         qsort(l, strv_length(l), sizeof(char*), str_compare);
         return l;
 }
+
+void strv_print(char **l) {
+        char **s;
+
+        if (!l)
+                return;
+
+        STRV_FOREACH(s, l)
+                puts(*s);
+}

diff --git a/src/shared/strv.h b/src/shared/strv.h
index fd728eff818b087cff05ec07b77c403aa94cf36b..33c752a313c0b36990c5e4da7825c49ec74aedff 100644 (file)
--- a/src/shared/strv.h
+++ b/src/shared/strv.h
@@ -87,3 +87,4 @@ bool strv_overlap(char **a, char **b);
 
 
 char **strv_sort(char **l);
+void strv_print(char **l);

diff --git a/src/test/test-env-replace.c b/src/test/test-env-replace.c
index cd596a6e168533ec84a32361a7a15aff0905fc94..ad5c9916085cf69473edd0c84f2e2eb1010d6c2a 100644 (file)
--- a/src/test/test-env-replace.c
+++ b/src/test/test-env-replace.c
@@ -58,10 +58,7 @@ int main(int argc, char *argv[]) {
         strv_free(a);
 
         r = replace_env_argv((char**) line, (char**) env);
-
-        STRV_FOREACH(i, r)
-                printf("%s\n", *i);
-
+        strv_print(r);
         strv_free(r);
 
         t = normalize_env_assignment("foo=bar");
@@ -127,16 +124,13 @@ int main(int argc, char *argv[]) {
         strv_free(a);
         strv_free(b);
 
-        STRV_FOREACH(i, r)
-                printf("%s\n", *i);
+        strv_print(r);
 
         printf("CLEANED UP:\n");
 
         r = strv_env_clean(r);
 
-        STRV_FOREACH(i, r)
-                printf("%s\n", *i);
-
+        strv_print(r);
         strv_free(r);
 
         return 0;

diff --git a/src/timedate/timedatectl.c b/src/timedate/timedatectl.c
index 281c0524da7d6eabfd3092885a6cbd3c32a34d6e..f5b5f0ca84e9accc702b1a4f8e4164de42ee8bd2 100644 (file)
--- a/src/timedate/timedatectl.c
+++ b/src/timedate/timedatectl.c
@@ -425,7 +425,6 @@ static int list_timezones(DBusConnection *bus, char **args, unsigned n) {
         _cleanup_fclose_ FILE *f = NULL;
         _cleanup_strv_free_ char **zones = NULL;
         size_t n_zones = 0;
-        char **i;
 
         assert(args);
         assert(n == 1);
@@ -487,8 +486,7 @@ static int list_timezones(DBusConnection *bus, char **args, unsigned n) {
         pager_open_if_enabled();
 
         strv_sort(zones);
-        STRV_FOREACH(i, zones)
-                puts(*i);
+        strv_print(zones);
 
         return 0;
 }