Thread (31 messages) 31 messages, 3 authors, 2015-06-30

Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement

From: Dan Carpenter <hidden>
Date: 2015-06-25 20:31:49
Also in: lkml

On Thu, Jun 25, 2015 at 08:49:30AM -0700, Juston Li wrote:
I placed it above the else to make it consistent with the if statement with
similar comment that the else corresponds to:

/* Set the primary display control */
- if (!ctrl)+ if (!ctrl) {
Once you add the else statement, then that kind of doesn't make sense.
Sudip is right.  It should be:

	} else {
		/* commentary about else side */

regards,
dan carpenter
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help