Re: [PATCH v3 01/38] sequencer: avoid unnecessary curly braces

Hi Junio,

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

Johannes Schindelin [off-list ref] writes:

quoted

 
-	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