Re: [greybus-dev] [Outreachy kernel] [PATCH v2 3/6] staging: greybus: Remove unnecessary braces

2017-02-21 Thread Alex Elder
On 02/21/2017 07:44 AM, Johan Hovold wrote: > On Tue, Feb 21, 2017 at 02:20:50PM +0100, Julia Lawall wrote: >> >> >> On Tue, 21 Feb 2017, simran singhal wrote: >> >>> This patch removes braces for single statement blocks. The warning >>> was detected using checkpatch.pl. >>> Coccinelle was used to

Re: [Outreachy kernel] [PATCH v2 3/6] staging: greybus: Remove unnecessary braces

2017-02-21 Thread Johan Hovold
On Tue, Feb 21, 2017 at 02:20:50PM +0100, Julia Lawall wrote: > > > On Tue, 21 Feb 2017, simran singhal wrote: > > > This patch removes braces for single statement blocks. The warning > > was detected using checkpatch.pl. > > Coccinelle was used to make the change. > > > > @@ > > expression e,e1

Re: [Outreachy kernel] [PATCH v2 3/6] staging: greybus: Remove unnecessary braces

2017-02-21 Thread Julia Lawall
On Tue, 21 Feb 2017, simran singhal wrote: > This patch removes braces for single statement blocks. The warning > was detected using checkpatch.pl. > Coccinelle was used to make the change. > > @@ > expression e,e1; > @@ > - if (e) { > + if (e) > e1; > - } > > Signed-off-by: simran singhal >

Re: [Outreachy kernel] [PATCH v2 3/6] staging: greybus: Remove unnecessary braces

2017-02-21 Thread Julia Lawall
On Tue, 21 Feb 2017, simran singhal wrote: > This patch removes braces for single statement blocks. The warning > was detected using checkpatch.pl. > Coccinelle was used to make the change. > > @@ > expression e,e1; > @@ > - if (e) { > + if (e) > e1; > - } > > Signed-off-by: simran singhal A