Re: [PATCH] checkpatch: add double empty line check

2012-11-17 Thread Eilon Greenstein
On Fri, 2012-11-16 at 16:50 -0800, Joe Perches wrote: Hi Joe, thanks for replying. > On Fri, 2012-11-16 at 22:04 +0200, Eilon Greenstein wrote: > > On Fri, 2012-11-16 at 11:55 -0800, David Rientjes wrote: > > > On Fri, 16 Nov 2012, Eilon Greenstein wrote: > > > This is fairly common in all the a

Re: [PATCH] checkpatch: add double empty line check

2012-11-16 Thread Joe Perches
On Fri, 2012-11-16 at 22:04 +0200, Eilon Greenstein wrote: > On Fri, 2012-11-16 at 11:55 -0800, David Rientjes wrote: > > On Fri, 16 Nov 2012, Eilon Greenstein wrote: > > > > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > > > index 21a9f5d..7a9c153 100755 > > > --- a/scripts/checkp

Re: [PATCH] checkpatch: add double empty line check

2012-11-16 Thread Eilon Greenstein
On Fri, 2012-11-16 at 11:55 -0800, David Rientjes wrote: > On Fri, 16 Nov 2012, Eilon Greenstein wrote: > > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > > index 21a9f5d..7a9c153 100755 > > --- a/scripts/checkpatch.pl > > +++ b/scripts/checkpatch.pl > > @@ -3579,6 +3579,13 @@ sub

Re: [PATCH] checkpatch: add double empty line check

2012-11-16 Thread David Rientjes
On Fri, 16 Nov 2012, Eilon Greenstein wrote: > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > index 21a9f5d..7a9c153 100755 > --- a/scripts/checkpatch.pl > +++ b/scripts/checkpatch.pl > @@ -3579,6 +3579,13 @@ sub process { > WARN("EXPORTED_WORLD_WRITABLE", >