From 3dfe7557809e5867306c7a0614b9d1c6036cbe4d Mon Sep 17 00:00:00 2001 From: Vaishali Thakkar Date: Fri, 19 Sep 2014 10:30:59 +0530 Subject: [PATCH] Staging: dgnc: Merge lines and remove unused variable for immediate return This patch merges two lines in a single line if immediate return is found. It also removes unnecessory variable rc as it is no longer needed. This is done using Coccinelle. Semantic patch used for this is as follows: @@ type T; identifier i; identifier f; constant C; @@ - T i; ...when != i when strict ( return -C; | - i = + return f(...); - return i; ) Signed-off-by: Vaishali Thakkar Reviewed-by: Julia Lawall Signed-off-by: Greg Kroah-Hartman --- drivers/staging/dgnc/dgnc_tty.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 0e903dc..a4e6c9e 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -2286,7 +2286,6 @@ static inline int dgnc_get_mstat(struct channel_t *ch) static int dgnc_get_modem_info(struct channel_t *ch, unsigned int __user *value) { int result; - int rc; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) return -ENXIO; @@ -2296,9 +2295,7 @@ static int dgnc_get_modem_info(struct channel_t *ch, unsigned int __user *value if (result < 0) return -ENXIO; - rc = put_user(result, value); - - return rc; + return put_user(result, value); } -- 2.7.4