[PATCH v2 9/9] list-objects-filter-options: make parser void

[Matthew DeVore]

This function always returns 0, so make it return void instead.

Signed-off-by: Matthew DeVore <matv...@google.com>
---
 list-objects-filter-options.c | 12 +++++-------
 list-objects-filter-options.h |  2 +-
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/list-objects-filter-options.c b/list-objects-filter-options.c
index d8abe6cfcf..ed02c88eb6 100644
--- a/list-objects-filter-options.c
+++ b/list-objects-filter-options.c
@@ -247,21 +247,21 @@ static void transform_to_combine_type(
        strbuf_release(&filter_options->sub[0].filter_spec);
 }
 
 void list_objects_filter_die_if_populated(
        struct list_objects_filter_options *filter_options)
 {
        if (filter_options->choice)
                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)
 {
        struct strbuf errbuf = STRBUF_INIT;
        int parse_error;
 
        if (!filter_options->choice) {
                strbuf_init(&filter_options->filter_spec, 0);
                strbuf_addstr(&filter_options->filter_spec, arg);
 
@@ -280,34 +280,32 @@ int parse_list_objects_filter(
                             filter_options->filter_spec.buf);
                ALLOC_GROW_BY(filter_options->sub, filter_options->sub_nr, 1,
                              filter_options->sub_alloc);
 
                parse_error = gently_parse_list_objects_filter(
                        &filter_options->sub[filter_options->sub_nr - 1], arg,
                        &errbuf);
        }
        if (parse_error)
                die("%s", errbuf.buf);
-       return 0;
 }
 
 int opt_parse_list_objects_filter(const struct option *opt,
                                  const char *arg, int unset)
 {
        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;
 }
 
 void expand_list_objects_filter_spec(
        const struct list_objects_filter_options *filter,
        struct strbuf *expanded_spec)
 {
        strbuf_init(expanded_spec, 0);
        if (filter->choice == LOFC_BLOB_LIMIT)
                strbuf_addf(expanded_spec, "blob:limit=%lu",
                            filter->blob_limit_value);
diff --git a/list-objects-filter-options.h b/list-objects-filter-options.h
index f8c8a624e4..2c0ce6383a 100644
--- a/list-objects-filter-options.h
+++ b/list-objects-filter-options.h
@@ -67,21 +67,21 @@ void list_objects_filter_die_if_populated(
        struct list_objects_filter_options *filter_options);
 
 /*
  * Parses the filter spec string given by arg and either (1) simply places the
  * result in filter_options if it is not yet populated or (2) combines it with
  * the filter already in filter_options if it is already populated. In the case
  * of (2), the filter specs are combined as if specified with 'combine:'.
  *
  * 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);
 
 int opt_parse_list_objects_filter(const struct option *opt,
                                  const char *arg, int unset);
 
 #define OPT_PARSE_LIST_OBJECTS_FILTER(fo) \
        { OPTION_CALLBACK, 0, CL_ARG__FILTER, fo, N_("args"), \
          N_("object filtering"), 0, \
          opt_parse_list_objects_filter }
-- 
2.17.1