summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLibravatar Matthew DeVore <matvore@google.com>2019-06-27 15:54:14 -0700
committerLibravatar Junio C Hamano <gitster@pobox.com>2019-06-28 08:41:53 -0700
commit90d21f9ebf6906f0ebb4fb1b20ec9536072e2916 (patch)
tree5a77a6dd1574984774484068ccdf507f68cd3c6f
parentlist-objects-filter-options: clean up use of ALLOC_GROW (diff)
downloadtgif-90d21f9ebf6906f0ebb4fb1b20ec9536072e2916.tar.xz
list-objects-filter-options: make parser void
This function always returns 0, so make it return void instead. Signed-off-by: Matthew DeVore <matvore@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--list-objects-filter-options.c12
-rw-r--r--list-objects-filter-options.h2
2 files changed, 6 insertions, 8 deletions
diff --git a/list-objects-filter-options.c b/list-objects-filter-options.c
index 44bc1153d1..ba1425cb4a 100644
--- a/list-objects-filter-options.c
+++ b/list-objects-filter-options.c
@@ -232,7 +232,7 @@ void list_objects_filter_die_if_populated(
die(_("multiple filter-specs cannot be combined"));
}
-int parse_list_objects_filter(
+void parse_list_objects_filter(
struct list_objects_filter_options *filter_options,
const char *arg)
{
@@ -262,7 +262,6 @@ int parse_list_objects_filter(
}
if (parse_error)
die("%s", errbuf.buf);
- return 0;
}
int opt_parse_list_objects_filter(const struct option *opt,
@@ -270,12 +269,11 @@ int opt_parse_list_objects_filter(const struct option *opt,
{
struct list_objects_filter_options *filter_options = opt->value;
- if (unset || !arg) {
+ if (unset || !arg)
list_objects_filter_set_no_filter(filter_options);
- return 0;
- }
-
- return parse_list_objects_filter(filter_options, arg);
+ else
+ parse_list_objects_filter(filter_options, arg);
+ return 0;
}
const char *list_objects_filter_spec(struct list_objects_filter_options *filter)
diff --git a/list-objects-filter-options.h b/list-objects-filter-options.h
index d8bc7e946e..db37dfb34a 100644
--- a/list-objects-filter-options.h
+++ b/list-objects-filter-options.h
@@ -74,7 +74,7 @@ void list_objects_filter_die_if_populated(
*
* Dies and prints a user-facing message if an error occurs.
*/
-int parse_list_objects_filter(
+void parse_list_objects_filter(
struct list_objects_filter_options *filter_options,
const char *arg);