staging: lustre: Remove unnecessary return capture
PATCH v3 1/1 staging: lustre: Remove unnecessary return capture
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;
---
Change in v2:
Make the commit message concise by changing 'return - call form of statement' to 'return statement'.
Change in v3:
Include changelog for changes in v2 and v3.
drivers/staging/lustre/lustre/mdc/mdc_request.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
drivers/staging/lustre/lustre/mdc/mdc_request.c +2 -3
@@ -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);
Signature · 1 line
2.7.4
Quoted 3 lines · Rehas Sachdeva
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:
Looks more readable, thanks.
julia
Quoted 49 lines
@@
local idexpression ret;
expression e;
@@(quoted diff) +1
-2-ret = +return e; -return ret;Signed-off-by Rehas Sachdeva <[email protected]>---
Change in v2:
Make the commit message concise by changing 'return - call form of statement' to 'return statement'.Change in v3:
Include changelog for changes in v2 and v3.drivers/staging/lustre/lustre/mdc/mdc_request.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)drivers/staging/lustre/lustre/mdc/mdc_request.c +2
-4@@ -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); --2.7.4
--
You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected].
To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20160915194108.GA5345%40toblerone.
For more options, visit https://groups.google.com/d/optout.