From 55d3426929d4d8c3dec402cabe6fb1bf27d6abad Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=86var=20Arnfj=C3=B6r=C3=B0=20Bjarmason?= Date: Thu, 22 Jun 2017 21:38:08 +0000 Subject: [PATCH] wildmatch: remove unused wildopts parameter MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove the unused wildopts placeholder struct from being passed to all wildmatch() invocations, or rather remove all the boilerplate NULL parameters. This parameter was added back in commit 9b3497cab9 ("wildmatch: rename constants and update prototype", 2013-01-01) as a placeholder for future use. Over 4 years later nothing has made use of it, let's just remove it. It can be added in the future if we find some reason to start using such a parameter. Signed-off-by: Ævar Arnfjörð Bjarmason Signed-off-by: Junio C Hamano --- apply.c | 2 +- builtin/describe.c | 4 ++-- builtin/ls-remote.c | 2 +- builtin/name-rev.c | 2 +- builtin/reflog.c | 2 +- builtin/replace.c | 2 +- builtin/show-branch.c | 2 +- config.c | 2 +- diffcore-order.c | 2 +- dir.c | 8 +++----- ref-filter.c | 4 ++-- refs.c | 2 +- revision.c | 2 +- t/helper/test-wildmatch.c | 6 +++--- wildmatch.c | 3 +-- wildmatch.h | 6 +----- 16 files changed, 22 insertions(+), 29 deletions(-) diff --git a/apply.c b/apply.c index 679ed7732d2..bb411b5810d 100644 --- a/apply.c +++ b/apply.c @@ -2077,7 +2077,7 @@ static int use_patch(struct apply_state *state, struct patch *p) /* See if it matches any of exclude/include rule */ for (i = 0; i < state->limit_by_name.nr; i++) { struct string_list_item *it = &state->limit_by_name.items[i]; - if (!wildmatch(it->string, pathname, 0, NULL)) + if (!wildmatch(it->string, pathname, 0)) return (it->util != NULL); } diff --git a/builtin/describe.c b/builtin/describe.c index 893c8789f4f..2fe05f5324e 100644 --- a/builtin/describe.c +++ b/builtin/describe.c @@ -142,7 +142,7 @@ static int get_name(const char *path, const struct object_id *oid, int flag, voi return 0; for_each_string_list_item(item, &exclude_patterns) { - if (!wildmatch(item->string, path + 10, 0, NULL)) + if (!wildmatch(item->string, path + 10, 0)) return 0; } } @@ -158,7 +158,7 @@ static int get_name(const char *path, const struct object_id *oid, int flag, voi return 0; for_each_string_list_item(item, &patterns) { - if (!wildmatch(item->string, path + 10, 0, NULL)) + if (!wildmatch(item->string, path + 10, 0)) break; /* If we get here, no pattern matched. */ diff --git a/builtin/ls-remote.c b/builtin/ls-remote.c index b2d7d5ce684..c4be98ab9e8 100644 --- a/builtin/ls-remote.c +++ b/builtin/ls-remote.c @@ -24,7 +24,7 @@ static int tail_match(const char **pattern, const char *path) pathbuf = xstrfmt("/%s", path); while ((p = *(pattern++)) != NULL) { - if (!wildmatch(p, pathbuf, 0, NULL)) { + if (!wildmatch(p, pathbuf, 0)) { free(pathbuf); return 1; } diff --git a/builtin/name-rev.c b/builtin/name-rev.c index 7fc7e66e850..d21a5002a7e 100644 --- a/builtin/name-rev.c +++ b/builtin/name-rev.c @@ -129,7 +129,7 @@ static int subpath_matches(const char *path, const char *filter) const char *subpath = path; while (subpath) { - if (!wildmatch(filter, subpath, 0, NULL)) + if (!wildmatch(filter, subpath, 0)) return subpath - path; subpath = strchr(subpath, '/'); if (subpath) diff --git a/builtin/reflog.c b/builtin/reflog.c index 920c16dac02..96c82b4f5ef 100644 --- a/builtin/reflog.c +++ b/builtin/reflog.c @@ -485,7 +485,7 @@ static void set_reflog_expiry_param(struct cmd_reflog_expire_cb *cb, int slot, c return; /* both given explicitly -- nothing to tweak */ for (ent = reflog_expire_cfg; ent; ent = ent->next) { - if (!wildmatch(ent->pattern, ref, 0, NULL)) { + if (!wildmatch(ent->pattern, ref, 0)) { if (!(slot & EXPIRE_TOTAL)) cb->expire_total = ent->expire_total; if (!(slot & EXPIRE_UNREACH)) diff --git a/builtin/replace.c b/builtin/replace.c index c921bc976f2..89262df5a19 100644 --- a/builtin/replace.c +++ b/builtin/replace.c @@ -40,7 +40,7 @@ static int show_reference(const char *refname, const struct object_id *oid, { struct show_data *data = cb_data; - if (!wildmatch(data->pattern, refname, 0, NULL)) { + if (!wildmatch(data->pattern, refname, 0)) { if (data->format == REPLACE_FORMAT_SHORT) printf("%s\n", refname); else if (data->format == REPLACE_FORMAT_MEDIUM) diff --git a/builtin/show-branch.c b/builtin/show-branch.c index 3636a05594d..82f378e7f59 100644 --- a/builtin/show-branch.c +++ b/builtin/show-branch.c @@ -437,7 +437,7 @@ static int append_matching_ref(const char *refname, const struct object_id *oid, slash--; if (!*tail) return 0; - if (wildmatch(match_ref_pattern, tail, 0, NULL)) + if (wildmatch(match_ref_pattern, tail, 0)) return 0; if (starts_with(refname, "refs/heads/")) return append_head_ref(refname, oid, flag, cb_data); diff --git a/config.c b/config.c index 34a139c40bd..260caf27b87 100644 --- a/config.c +++ b/config.c @@ -245,7 +245,7 @@ static int include_by_gitdir(const struct config_options *opts, } ret = !wildmatch(pattern.buf + prefix, text.buf + prefix, - icase ? WM_CASEFOLD : 0, NULL); + icase ? WM_CASEFOLD : 0); if (!ret && !already_tried_absolute) { /* diff --git a/diffcore-order.c b/diffcore-order.c index 1957f822a51..19e73311f9c 100644 --- a/diffcore-order.c +++ b/diffcore-order.c @@ -67,7 +67,7 @@ static int match_order(const char *path) strbuf_addstr(&p, path); while (p.buf[0]) { char *cp; - if (!wildmatch(order[i], p.buf, 0, NULL)) + if (!wildmatch(order[i], p.buf, 0)) return i; cp = strrchr(p.buf, '/'); if (!cp) diff --git a/dir.c b/dir.c index 5f1afb56bc6..c64965f2374 100644 --- a/dir.c +++ b/dir.c @@ -91,13 +91,11 @@ int git_fnmatch(const struct pathspec_item *item, if (item->magic & PATHSPEC_GLOB) return wildmatch(pattern, string, WM_PATHNAME | - (item->magic & PATHSPEC_ICASE ? WM_CASEFOLD : 0), - NULL); + (item->magic & PATHSPEC_ICASE ? WM_CASEFOLD : 0)); else /* wildmatch has not learned no FNM_PATHNAME mode yet */ return wildmatch(pattern, string, - item->magic & PATHSPEC_ICASE ? WM_CASEFOLD : 0, - NULL); + item->magic & PATHSPEC_ICASE ? WM_CASEFOLD : 0); } static int fnmatch_icase_mem(const char *pattern, int patternlen, @@ -121,7 +119,7 @@ static int fnmatch_icase_mem(const char *pattern, int patternlen, if (ignore_case) flags |= WM_CASEFOLD; - match_status = wildmatch(use_pat, use_str, flags, NULL); + match_status = wildmatch(use_pat, use_str, flags); strbuf_release(&pat_buf); strbuf_release(&str_buf); diff --git a/ref-filter.c b/ref-filter.c index ab32bc9c314..718def443a0 100644 --- a/ref-filter.c +++ b/ref-filter.c @@ -1624,7 +1624,7 @@ static int match_pattern(const struct ref_filter *filter, const char *refname) skip_prefix(refname, "refs/", &refname)); for (; *patterns; patterns++) { - if (!wildmatch(*patterns, refname, flags, NULL)) + if (!wildmatch(*patterns, refname, flags)) return 1; } return 0; @@ -1655,7 +1655,7 @@ static int match_name_as_path(const struct ref_filter *filter, const char *refna refname[plen] == '/' || p[plen-1] == '/')) return 1; - if (!wildmatch(p, refname, WM_PATHNAME, NULL)) + if (!wildmatch(p, refname, WM_PATHNAME)) return 1; } return 0; diff --git a/refs.c b/refs.c index f0685c92513..32626a4cd14 100644 --- a/refs.c +++ b/refs.c @@ -229,7 +229,7 @@ static int filter_refs(const char *refname, const struct object_id *oid, { struct ref_filter *filter = (struct ref_filter *)data; - if (wildmatch(filter->pattern, refname, 0, NULL)) + if (wildmatch(filter->pattern, refname, 0)) return 0; return filter->fn(refname, oid, flags, filter->cb_data); } diff --git a/revision.c b/revision.c index d81c08a1dee..4ab8344ee87 100644 --- a/revision.c +++ b/revision.c @@ -1142,7 +1142,7 @@ int ref_excluded(struct string_list *ref_excludes, const char *path) if (!ref_excludes) return 0; for_each_string_list_item(item, ref_excludes) { - if (!wildmatch(item->string, path, 0, NULL)) + if (!wildmatch(item->string, path, 0)) return 1; } return 0; diff --git a/t/helper/test-wildmatch.c b/t/helper/test-wildmatch.c index 52be876fed3..921d7b3e7ea 100644 --- a/t/helper/test-wildmatch.c +++ b/t/helper/test-wildmatch.c @@ -11,11 +11,11 @@ int cmd_main(int argc, const char **argv) argv[i] += 3; } if (!strcmp(argv[1], "wildmatch")) - return !!wildmatch(argv[3], argv[2], WM_PATHNAME, NULL); + return !!wildmatch(argv[3], argv[2], WM_PATHNAME); else if (!strcmp(argv[1], "iwildmatch")) - return !!wildmatch(argv[3], argv[2], WM_PATHNAME | WM_CASEFOLD, NULL); + return !!wildmatch(argv[3], argv[2], WM_PATHNAME | WM_CASEFOLD); else if (!strcmp(argv[1], "pathmatch")) - return !!wildmatch(argv[3], argv[2], 0, NULL); + return !!wildmatch(argv[3], argv[2], 0); else return 1; } diff --git a/wildmatch.c b/wildmatch.c index 57c87658059..d074c1be104 100644 --- a/wildmatch.c +++ b/wildmatch.c @@ -272,8 +272,7 @@ static int dowild(const uchar *p, const uchar *text, unsigned int flags) } /* Match the "pattern" against the "text" string. */ -int wildmatch(const char *pattern, const char *text, - unsigned int flags, struct wildopts *wo) +int wildmatch(const char *pattern, const char *text, unsigned int flags) { return dowild((const uchar*)pattern, (const uchar*)text, flags); } diff --git a/wildmatch.h b/wildmatch.h index 4090c8f4bb0..b8c826aa684 100644 --- a/wildmatch.h +++ b/wildmatch.h @@ -10,9 +10,5 @@ #define WM_ABORT_ALL -1 #define WM_ABORT_TO_STARSTAR -2 -struct wildopts; - -int wildmatch(const char *pattern, const char *text, - unsigned int flags, - struct wildopts *wo); +int wildmatch(const char *pattern, const char *text, unsigned int flags); #endif