[PATCH 1/3] pretty.c: rename describe options variable to more descriptive name
- From
Eli Schwartz <eschwartz@archlinux.org>
- Date
- Oct 24, 2021, 01:42 UTC
- Message-ID
- <20211024014256.3569322-2-eschwartz@archlinux.org>
- In-Reply-To
- <20211024014256.3569322-1-eschwartz@archlinux.org>
It contains option arguments, not options. We would like to add option support here too.
Signed-off-by: Eli Schwartz <eschwartz@archlinux.org> --- pretty.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/pretty.c b/pretty.c index 73b5ead509..9db2c65538 100644 --- a/pretty.c +++ b/pretty.c @@ -1216,7 +1216,7 @@ int format_set_trailers_options(struct process_trailer_options *opts, static size_t parse_describe_args(const char *start, struct strvec *args) { - const char *options[] = { "match", "exclude" }; + const char *option_arguments[] = { "match", "exclude" }; const char *arg = start; for (;;) { @@ -1225,10 +1225,10 @@ static size_t parse_describe_args(const char *start, struct strvec *args) size_t arglen = 0; int i; - for (i = 0; i < ARRAY_SIZE(options); i++) { - if (match_placeholder_arg_value(arg, options[i], &arg, + for (i = 0; i < ARRAY_SIZE(option_arguments); i++) { + if (match_placeholder_arg_value(arg, option_arguments[i], &arg, &argval, &arglen)) { - matched = options[i]; + matched = option_arguments[i]; break; } }
-- 2.33.1