[PATCH] staging: rtl8192u: remove blank lines after braces (opening)

Andy Shevchenko andy.shevchenko at gmail.com
Thu Apr 7 18:06:37 UTC 2016


On Thu, Apr 7, 2016 at 9:04 PM, Joe Perches <joe at perches.com> wrote:
> On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote:
>> On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim <nicholassimws at gmail.com> wrote:
>> >
>> > Remove unneeded blank lines appearing after opening braces as suggested
>> > by checkpatch.pl
>> >
>> You have to combine this with the rest so called 'indentation' fixes,
>> e.g. +- empty lines, tabs vs. spaces, etc.
>
> Actually, patches that do single things like removing
> undesired blank lines are easier to review if separate
> from patches that change horizontal whitespace.
>

Okay, but I think you at least agree that +- empty lines can be easily combined.


-- 
With Best Regards,
Andy Shevchenko


More information about the devel mailing list