Messages in this thread | | | Date | Wed, 9 Jul 2014 04:12:15 +0200 | From | Fabian Frederick <> | Subject | Re: [PATCH 1/1] dvb-frontends: remove unnecessary break after goto |
| |
On Tue, 08 Jul 2014 21:35:58 +0300 Antti Palosaari <crope@iki.fi> wrote:
> Moikka Fabian! > I have no reason to decline that patch (I will apply it) even it has > hardly meaning. But is there now some new tool which warns that kind of > issues? Hello Antti,
Thanks :) AFAIK there's still no automatic detection of those cases.
Regards, Fabian > > regards > Atnti > > > On 07/08/2014 08:23 PM, Fabian Frederick wrote: > > Cc: Antti Palosaari <crope@iki.fi> > > Cc: Mauro Carvalho Chehab <m.chehab@samsung.com> > > Cc: linux-media@vger.kernel.org > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > > --- > > drivers/media/dvb-frontends/af9013.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/media/dvb-frontends/af9013.c b/drivers/media/dvb-frontends/af9013.c > > index fb504f1..ecf6388 100644 > > --- a/drivers/media/dvb-frontends/af9013.c > > +++ b/drivers/media/dvb-frontends/af9013.c > > @@ -470,7 +470,6 @@ static int af9013_statistics_snr_result(struct dvb_frontend *fe) > > break; > > default: > > goto err; > > - break; > > } > > > > for (i = 0; i < len; i++) { > > > > -- > http://palosaari.fi/
| |