Subject: Re: [Patch] Nice help Output - SmallProjectIdea[1]

From: Thomas Gummerer <[email-masked]>
Date: Fri, 01 May 2015 21:11:12 +0200

Hi,

Alangi Derick <[email-masked]> writes:

> This is a patch to fix the problem as given in the site:
> https://git.wiki.kernel.org/index.php/SmallProjectsIdeas

Please have a look at Documentation/SubmittingPatches, on how to submit
well formed patches.  In particular, the following things should be
improved:

- The commit should have a meaningful commit message (See section 2 in
  SubmittingPatches)
- The patch should be created with git format-patch, so it can be
  applied using git am. (See section 3 and 4)
- Your work should be signed off, otherwise it can not be applied to the
  project. (See section 5)

> diff --git a/help.c b/help.c
> index 2072a87..e347bbc 100644
> --- a/help.c
> +++ b/help.c
> @@ -229,9 +229,18 @@ void list_common_cmds_help(void)
>
>   puts(_("The most commonly used git commands are:"));
>   for (i = 0; i < ARRAY_SIZE(common_cmds); i++) {
> - printf("   %s   ", common_cmds[i].name);
> - mput_char(' ', longest - strlen(common_cmds[i].name));
> - puts(_(common_cmds[i].help));
> + if(i%3 == 1){
> + printf("   %s   ", common_cmds[i].name);
> + mput_char(' ', longest - strlen(common_cmds[i].name));
> + puts(_(common_cmds[i].help));
> + printf("\n");
> + }
> + else {
> + printf("   %s   ", common_cmds[i].name);
> + mput_char(' ', longest - strlen(common_cmds[i].name));
> + puts(_(common_cmds[i].help));
> + }
> +

Patch formatting issues aside, I don't think this does what is suggested
in the project.  You are just adding a blank line after three lines in
the help output, which I don't think improves the current state by much.

The bzr help output on the other hand groups similar commands together,
and shows them to the user in groups.  So the first step would be to
find out which commands should be grouped together and then to output
the groups here.

>   }
>  }
>
>
> Regards
> Alangi Derick Ndimnain
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to [email-masked]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html