basic/strv: drop flags argument from strv_fnmatch()

diff --git a/src/analyze/analyze.c b/src/analyze/analyze.c
index 991e61de7e7ff99a90fbc4bd60205379717ed85f..9674a8af87d3923ac0f778206225d52ac7401369 100644 (file)
--- a/src/analyze/analyze.c
+++ b/src/analyze/analyze.c
@@ -1162,9 +1162,9 @@ static int graph_one_property(
         assert(prop);
         assert(color);
 
-        match_patterns = strv_fnmatch(patterns, u->id, 0);
+        match_patterns = strv_fnmatch(patterns, u->id);
 
-        if (!strv_isempty(from_patterns) && !match_patterns && !strv_fnmatch(from_patterns, u->id, 0))
+        if (!strv_isempty(from_patterns) && !match_patterns && !strv_fnmatch(from_patterns, u->id))
                 return 0;
 
         r = bus_get_unit_property_strv(bus, u->unit_path, prop, &units);
@@ -1174,9 +1174,9 @@ static int graph_one_property(
         STRV_FOREACH(unit, units) {
                 bool match_patterns2;
 
-                match_patterns2 = strv_fnmatch(patterns, *unit, 0);
+                match_patterns2 = strv_fnmatch(patterns, *unit);
 
-                if (!strv_isempty(to_patterns) && !match_patterns2 && !strv_fnmatch(to_patterns, *unit, 0))
+                if (!strv_isempty(to_patterns) && !match_patterns2 && !strv_fnmatch(to_patterns, *unit))
                         continue;
 
                 if (!strv_isempty(patterns) && !match_patterns && !match_patterns2)

diff --git a/src/basic/strv.h b/src/basic/strv.h
index 1cae0ae09fd8453055b1e9fda8092a7b7e44ade6..16faf9e7f25ba57a32151999c21fcf1b74e0fed9 100644 (file)
--- a/src/basic/strv.h
+++ b/src/basic/strv.h
@@ -178,14 +178,14 @@ char **strv_reverse(char **l);
 char **strv_shell_escape(char **l, const char *bad);
 
 bool strv_fnmatch_full(char* const* patterns, const char *s, int flags, size_t *matched_pos);
-static inline bool strv_fnmatch(char* const* patterns, const char *s, int flags) {
-        return strv_fnmatch_full(patterns, s, flags, NULL);
+static inline bool strv_fnmatch(char* const* patterns, const char *s) {
+        return strv_fnmatch_full(patterns, s, 0, NULL);
 }
 
 static inline bool strv_fnmatch_or_empty(char* const* patterns, const char *s, int flags) {
         assert(s);
         return strv_isempty(patterns) ||
-               strv_fnmatch(patterns, s, flags);
+               strv_fnmatch_full(patterns, s, flags, NULL);
 }
 
 char ***strv_free_free(char ***l);

diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c
index d9a695b3a41573d9990d070c8a87b9dfb4011b17..07c6cf345109e912e7f649b386fe7412888b0954 100644 (file)
--- a/src/network/networkd-link.c
+++ b/src/network/networkd-link.c
@@ -1875,7 +1875,7 @@ static int link_new_bound_by_list(Link *link) {
                 if (strv_isempty(carrier->network->bind_carrier))
                         continue;
 
-                if (strv_fnmatch(carrier->network->bind_carrier, link->ifname, 0)) {
+                if (strv_fnmatch(carrier->network->bind_carrier, link->ifname)) {
                         r = link_put_carrier(link, carrier, &link->bound_by_links);
                         if (r < 0)
                                 return r;
@@ -1917,7 +1917,7 @@ static int link_new_bound_to_list(Link *link) {
         m = link->manager;
 
         HASHMAP_FOREACH (carrier, m->links, i) {
-                if (strv_fnmatch(link->network->bind_carrier, carrier->ifname, 0)) {
+                if (strv_fnmatch(link->network->bind_carrier, carrier->ifname)) {
                         r = link_put_carrier(link, carrier, &link->bound_to_links);
                         if (r < 0)
                                 return r;

diff --git a/src/network/wait-online/manager.c b/src/network/wait-online/manager.c
index 6de3df94e1f2b2334ae4c48805224b6258a2df9c..0e6a284da2f2dcbf4370c74a2c9013c56bc1bb29 100644 (file)
--- a/src/network/wait-online/manager.c
+++ b/src/network/wait-online/manager.c
@@ -29,7 +29,7 @@ static bool manager_ignore_link(Manager *m, Link *link) {
                 return true;
 
         /* ignore interfaces we explicitly are asked to ignore */
-        return strv_fnmatch(m->ignore, link->ifname, 0);
+        return strv_fnmatch(m->ignore, link->ifname);
 }
 
 static int manager_link_is_online(Manager *m, Link *l, LinkOperationalState s) {

diff --git a/src/test/test-strv.c b/src/test/test-strv.c
index b8001dfe70aaebd113495ff73e46a35e7a50a0d8..b63cb4b63e9d7bec5b8d8b0a48992eca943a1a0a 100644 (file)
--- a/src/test/test-strv.c
+++ b/src/test/test-strv.c
@@ -928,7 +928,7 @@ static void test_strv_fnmatch(void) {
 
         log_info("/* %s */", __func__);
 
-        assert_se(!strv_fnmatch(STRV_MAKE_EMPTY, "a", 0));
+        assert_se(!strv_fnmatch(STRV_MAKE_EMPTY, "a"));
 
         v = strv_new("xxx", "*\\*", "yyy");
         assert_se(!strv_fnmatch_full(v, "\\", 0, NULL));