Re: [PATCH] checkpatch: Add control statement test to SINGLE_STATEMENT_DO_WHILE_MACRO

2012-08-02 Thread Stephen Rothwell
Hi Franz, On Thu, 02 Aug 2012 18:44:57 +0200 Schrober wrote: > > On Thursday 02 August 2012 17:23:22 Andy Whitcroft wrote: > > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > > > index 913d6bd..ca05ba2 100755 > > > --- a/scripts/checkpatch.pl > > > +++ b/scripts/checkpatch.pl > > >

Re: [PATCH] checkpatch: Add control statement test to SINGLE_STATEMENT_DO_WHILE_MACRO

2012-08-02 Thread Schrober
On Thursday 02 August 2012 17:23:22 Andy Whitcroft wrote: > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > > index 913d6bd..ca05ba2 100755 > > --- a/scripts/checkpatch.pl > > +++ b/scripts/checkpatch.pl > > @@ -3016,7 +3016,8 @@ sub process { > > > >

Re: [PATCH] checkpatch: Add control statement test to SINGLE_STATEMENT_DO_WHILE_MACRO

2012-08-02 Thread Andy Whitcroft
On Thu, Aug 02, 2012 at 08:26:53AM -0700, Joe Perches wrote: > The patch below is tested with: > $ cat t.c > > #define bar_1(foo)\ > if (foo)\ > baz(); \ > > #define bar_2(foo)