Re: [Cocci] [PATCH 2/4] coccinelle: platform_get_irq: handle 2-statement branches

2019-11-19 Thread Julia Lawall
On Tue, 19 Nov 2019, Markus Elfring wrote: > > From: Masahiro Yamada > > I wonder about this information. > Would you like to use the tag “Suggested-by” instead? Sorry, I seem to have done something quite wrong on this patch. I will fix it. > > > … > > +++ b/scripts/coccinelle/api/platform_g

Re: [Cocci] [PATCH 2/4] coccinelle: platform_get_irq: handle 2-statement branches

2019-11-19 Thread Markus Elfring
> From: Masahiro Yamada I wonder about this information. Would you like to use the tag “Suggested-by” instead? … > +++ b/scripts/coccinelle/api/platform_get_irq.cocci > @@ -31,6 +31,25 @@ if ( \( ret < 0 \| ret <= 0 \) ) … > +ret = > +( > +platform_get_irq > +| > +platform_get_irq_byname > +)(E

Re: [Cocci] [PATCH 2/4] coccinelle: platform_get_irq: handle 2-statement branches

2019-11-19 Thread Markus Elfring
> From: Masahiro Yamada I wonder about this information. Would you like to use the tag “Suggested-by” instead? … > +++ b/scripts/coccinelle/api/platform_get_irq.cocci > @@ -31,6 +31,25 @@ if ( \( ret < 0 \| ret <= 0 \) ) … > +ret = > +( > +platform_get_irq > +| > +platform_get_irq_byname > +)(E

[Cocci] [PATCH 2/4] coccinelle: platform_get_irq: handle 2-statement branches

2019-11-19 Thread Julia Lawall
From: Masahiro Yamada Treat separately the case where there is only one other statement in the branch, to be able to remove the outer {} as well. Signed-off-by: Julia Lawall --- scripts/coccinelle/api/platform_get_irq.cocci | 19 +++ 1 file changed, 19 insertions(+) diff --