Re: [PATCH] usage_with_options: omit double new line on empty option list
Stefan Beller <sbeller@google.com> writes:
Show 18 quoted lines
> Currently the worktree command gives its usage, when no subcommand is
> given. However there are no general options, all options are related to
> the subcommands itself, such that:
>
> $ git worktree
> usage: git worktree add [<options>] <path> [<branch>]
> or: git worktree list [<options>]
> or: git worktree lock [<options>] <path>
> or: git worktree prune [<options>]
> or: git worktree unlock <path>
>
>
> $
>
> Note the two empty lines at the end of the usage string. This is because
> the toplevel usage is printed with an empty options list.
>
> Only print one new line after the usage string if the option list is empty.
Good find. Shouldn't the last word in the sentence "non-empty", though?
Show 19 quoted lines
>
> Signed-off-by: Stefan Beller <sbeller@google.com>
> ---
> parse-options.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/parse-options.c b/parse-options.c
> index 0dd9fc6a0d..1307c82861 100644
> --- a/parse-options.c
> +++ b/parse-options.c
> @@ -603,7 +603,7 @@ static int usage_with_options_internal(struct parse_opt_ctx_t *ctx,
> usagestr++;
> }
>
> - if (opts->type != OPTION_GROUP)
> + if (opts->type != OPTION_GROUP && opts->type != OPTION_END)
> fputc('\n', outfile);
>
> for (; opts->type != OPTION_END; opts++) {Junio C Hamano <gitster@pobox.com>