Messages in this thread | | | Date | Thu, 25 Jun 2015 23:31:15 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement |
| |
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
| |