Hi Junio,

On Thu, 12 Jan 2017, Junio C Hamano wrote:

> Johannes Schindelin <johannes.schinde...@gmx.de> writes:
> 
> >  
> > -   if (!commit->parents) {
> > +   if (!commit->parents)
> >             parent = NULL;
> > -   }
> >     else if (commit->parents->next) {
> >             /* Reverting or cherry-picking a merge commit */
> >             int cnt;
> 
> The result becomes
> 
>       if (...)
>               single statement;
>       else if (...) {
>               multiple;
>                 statements;
>         }
> 
> which is not quite an improvement.  

Yet, this used to be the coding style of Git, and your statement comes
quite as a surprise to me.

Ciao,
Johannes

Reply via email to