redonkable/alistair23-linux
redonkable/alistair23-linux
1
0
Fork 0
Code Releases Activity

Staging: lustre: Unnecessory else and braces are removed

Browse source 
This patch fixes following checkpatch.pl warning using coccinelle:

WARNING: else is not generally useful after a break or return

Semantic patch used for this is as follows:

@rule1@
expression e1;
@@

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

@rule2@
expression e2;
statement s1;
@@

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

Signed-off-by: Vaishali Thakkar <vthakkar1994@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Vaishali Thakkar 2014-09-23 19:21:32 +05:30 committed by Greg Kroah-Hartman
parent 5ce91a9e85
commit 2d02b0b81c
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
drivers/staging/lustre/lustre/obdclass/capa.c
View file

@ -213,12 +213,11 @@ struct obd_capa *capa_add(struct hlist_head *hash, struct lustre_capa *capa)
capa_delete_lru(list);
spin_unlock(&capa_lock);
return ocapa;
} else {
capa_get(old);
spin_unlock(&capa_lock);
capa_put(ocapa);
return old;
}
capa_get(old);
spin_unlock(&capa_lock);
capa_put(ocapa);
return old;
}
EXPORT_SYMBOL(capa_add);