Messages in this thread | | | Date | Tue, 14 Jan 2014 10:14:25 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] Staging: comedi: remove unnecessary braces in pcl711.c |
| |
On Mon, Jan 13, 2014 at 09:13:16PM -0600, Chase Southwood wrote: > This patch for pcl711.c removes braces causing a checkpatch.pl warning. > It also removes an empty else arm of an if-else statement. > > Signed-off-by: Chase Southwood <chase.southwood@yahoo.com> > --- > > I removed the "else" arm of this statement because it was empty, save > for the "ignore" comment. If it is instead preferred kernel style for > this empty else to remain in, let me know and I will correct this > patch accordingly. >
Nope, it's perfect. Don't doubt yourself. Always feel free to change the code.
regards, dan carpenter
| |