staging: lustre: Remove unnecessary return capture
authorRehas Sachdeva <aquannie@gmail.com>
Thu, 15 Sep 2016 19:41:08 +0000 (01:11 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 16 Sep 2016 08:16:15 +0000 (10:16 +0200)
Instead of storing the return value of a function call into a variable and
then returning it, we can club the two into a single return statement. This
change was made using the following semantic patch by Coccinelle:

@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;

Signed-off-by: Rehas Sachdeva <aquannie@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/mdc/mdc_request.c

index 5bf95f9..cf94926 100644 (file)
@@ -2488,9 +2488,8 @@ static int mdc_set_info_async(const struct lu_env *env,
                }
                spin_unlock(&imp->imp_lock);
 
-               rc = do_set_info_async(imp, MDS_SET_INFO, LUSTRE_MDS_VERSION,
-                                      keylen, key, vallen, val, set);
-               return rc;
+               return do_set_info_async(imp, MDS_SET_INFO, LUSTRE_MDS_VERSION,
+                                        keylen, key, vallen, val, set);
        }
        if (KEY_IS(KEY_SPTLRPC_CONF)) {
                sptlrpc_conf_client_adapt(exp->exp_obd);