On Mon, Feb 17, 2014 at 12:45:07PM -0800, Chase Southwood wrote: > >On Monday, February 17, 2014 7:16 AM, Ian Abbott <abbotti@xxxxxxxxx> wrote: > > >>On 2014-02-16 08:40, Chase Southwood wrote: > >> This patch for hwdrv_apci035 removes some unneeded braces, and moves some > >> improperly placed braces to the correct position, as found by checkpatch. > >> It also removes a commented out if-statement that I found whilst cleaning > >> braces that is identical to another un-commented if-statement directly > >> above it, so it is just added clutter and so we can delete it to clean up > >> further. > >> > >> Signed-off-by: Chase Southwood <chase.southwood@xxxxxxxxx> > >> --- > >> So I decided to venture into addi-data today and found that most of the > >> files in there are very messy from a style standpoint. This is the first > >> (of probably a few) patchsets to try and clean those files up a bit. I > >> hope that this will be helpful! > > > >Quite a few have been cleaned up extensively by Hartley, but they needed > >more extensive changes than clean-ups due to them trying to handle > >things differently to the normal comedi way of doing things. > > Oh, I see. Makes perfect sense. Well like I said, if cleaning them > up a bit will help make reworking them any easier or seems like useful > work, I'd be happy to continue making them a little easier on the > eyes! Ian acked your patch already... If you want to clean these up then go for it. There is no reason Hartley should have to do all the work. But consider doing more "extensive" cleanups. Instead of just shifting the comments over in [patch 2/4] you could change the format to kernel style. Newbies are too timid. Look through the work that Hartley has done and try copy it. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
