Re: [PATCHv2 3/5] staging:android:ion: Fix blank line issues
From: Greg KH
Date: Fri Sep 23 2016 - 11:06:42 EST
On Fri, Sep 23, 2016 at 04:26:28PM +0300, Yannis Damigos wrote:
> On Fri, Sep 23, 2016 at 3:17 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Fri, Sep 23, 2016 at 11:05:15AM +0300, giannis.damigos@xxxxxxxxx wrote:
> >> From: Yannis Damigos <giannis.damigos@xxxxxxxxx>
> >>
> >> This patch fixes the following blank line issues:
> >>
> >> CHECK: Please don't use multiple blank lines
> >> +
> >> +
> >
> > What does that mean with the + + lines?
> >
> >>
> >> CHECK: Please use a blank line after function/struct/union/enum
> >> declarations
> >> +}
> >> +RESERVEDMEM_OF_DECLARE(ion, "ion-region", rmem_ion_setup);
> >
> > Linewrap :(
> >
> If the line is not wrapped, I get the following warning:
> Possible unwrapped commit description (prefer a maximum 75 chars per line)
>
> Should I ignore it?
Don't you think it makes sense to do so? :)