staging: comedi: drivers: Remove unnecessary else following return
authorJanani Ravichandran <janani.rvchndrn@gmail.com>
Thu, 18 Feb 2016 22:48:17 +0000 (17:48 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Feb 2016 23:25:58 +0000 (15:25 -0800)
Else is unnecessary when there is a return statement in the
corresponding if block.
Coccinelle patch:

@rule1@
expression e1;
@@

if (e1) { ... return ...; }
-       else{
        ...
-            }

@rule2@
expression e2;
statement s1;
@@

if(e2) { ... return ...; }
-       else
s1

Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/comedi_isadma.c

index 6ba71d114a95c180f95eebb6d1af93707cd50818..68ef9b1750be8adefc21d148d949588969fda6c5 100644 (file)
@@ -132,8 +132,7 @@ unsigned int comedi_isadma_poll(struct comedi_isadma *dma)
                result = result1;
        if (result >= desc->size || result == 0)
                return 0;
-       else
-               return desc->size - result;
+       return desc->size - result;
 }
 EXPORT_SYMBOL_GPL(comedi_isadma_poll);