Subject: Re: [PATCH v2 4/9] help.c: split up list_all_cmds_help() function

From: Junio C Hamano <[email-masked]>
Date: Wed, 23 Feb 2022 14:03:28 -0800

Ævar Arnfjörð Bjarmason  <[email-masked]> writes:

> -void list_all_cmds_help(void)
> +static void list_all_cmds_help_external_commands(void)
>  {
>  	struct string_list others = STRING_LIST_INIT_DUP;
> -	struct string_list alias_list = STRING_LIST_INIT_DUP;
> -	struct cmdname_help *aliases;
> -	int i, longest;
> -
> -	puts(_("See 'git help <command>' to read about a specific subcommand"));
> -	print_cmd_by_category(main_categories, &longest);
> +	int i;
>  
>  	list_all_other_cmds(&others);
>  	if (others.nr)

Let's note that in the body of this new helper function, we still
use the printf("\n%s\n", _("group header")) pattern, not putchar('\n')
followed by puts().

> @@ -449,6 +444,13 @@ void list_all_cmds_help(void)
>  	for (i = 0; i < others.nr; i++)
>  		printf("   %s\n", others.items[i].string);
>  	string_list_clear(&others, 0);
> +}
> +
> +static void list_all_cmds_help_aliases(int longest)
> +{
> +	struct string_list alias_list = STRING_LIST_INIT_DUP;
> +	struct cmdname_help *aliases;
> +	int i;
>  
>  	git_config(get_alias, &alias_list);
>  	string_list_sort(&alias_list);

And this helper, too.

> @@ -474,6 +476,17 @@ void list_all_cmds_help(void)
>  	string_list_clear(&alias_list, 1);
>  }
>  
> +void list_all_cmds_help(void)
> +{
> +	int longest;
> +
> +	puts(_("See 'git help <command>' to read about a specific subcommand"));
> +	print_cmd_by_category(main_categories, &longest);
> +
> +	list_all_cmds_help_external_commands();
> +	list_all_cmds_help_aliases(longest);
> +}

This does make sense ;-)