On 11/20, Jonathan Nieder wrote:
> Brandon Williams wrote:
> > On 11/20, Jonathan Nieder wrote:
> 
> >> +  /* These underlying connection commands die() if they
> >> +   * cannot connect.
> >> +   */
> >
> > I know this is really just code motion but maybe we can fix the style of
> > the comment here?
> 
> How about doing that as a separate commit?

Looks good!

> 
> -- >8 --
> Subject: connect: correct style of C-style comment
> 
> Documentation/CodingGuidelines explains:
> 
>  - Multi-line comments include their delimiters on separate lines from
>    the text.  E.g.
> 
>       /*
>        * A very long
>        * multi-line comment.
>        */
> 
> Reported-by: Brandon Williams <[email protected]>
> Signed-off-by: Jonathan Nieder <[email protected]>
> ---
>  connect.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git i/connect.c w/connect.c
> index 20ed1d9574..e544a5e1dd 100644
> --- i/connect.c
> +++ w/connect.c
> @@ -889,7 +889,8 @@ static struct child_process *git_connect_git(int fd[2], 
> char *hostandport,
>  
>       transport_check_allowed("git");
>  
> -     /* These underlying connection commands die() if they
> +     /*
> +      * These underlying connection commands die() if they
>        * cannot connect.
>        */
>       if (git_use_proxy(hostandport))

-- 
Brandon Williams

Reply via email to