Received-SPF: pass (google.com: domain of anchalj109@gmail.com designates 2a00:1450:400c:c09::22e as permitted sender) client-ip=2a00:1450:400c:c09::22e;
Date: Sun, 1 Nov 2015 19:59:32 +0530
From: Anchal Jain <anchalj109@gmail.com>
To: outreachy-kernel@googlegroups.com
Subject: [PATCH] staging: lustre: lnet: Remove an else after an if-return
Message-ID: <20151101142932.GA12081@anchal-VirtualBox>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.5.21 (2010-09-15)

The original motivation of the patch was to remove an 
else after an if branch that ends in a return. This 
removal required the removal of a list of other things. 
That are needed to make the code either correct or 
acceptable to checkpatch, for example:removing a blank 
line, removing braces, etc.

Remove unnecessary use of else that required the
removal of corresponding braces. And then the
declaration of hash was movedup to the beginning of the
function is required, but the initialization was left
where it was. And it also required remove the blank
line and add some new lines before the if and below the
declaration of hash.

Signed-off-by: Anchal Jain <anchalj109@gmail.com>

---
 drivers/staging/lustre/lnet/lnet/lib-ptl.c | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/lustre/lnet/lnet/lib-ptl.c b/drivers/staging/lustre/lnet/lnet/lib-ptl.c
index b4f573a..aec06f9 100644
--- a/drivers/staging/lustre/lnet/lnet/lib-ptl.c
+++ b/drivers/staging/lustre/lnet/lnet/lib-ptl.c
@@ -357,16 +357,14 @@ lnet_mt_match_head(struct lnet_match_table *mtable,
 		   lnet_process_id_t id, __u64 mbits)
 {
 	struct lnet_portal *ptl = the_lnet.ln_portals[mtable->mt_portal];
-
-	if (lnet_ptl_is_wildcard(ptl)) {
+	 unsigned long hash;
+	if (lnet_ptl_is_wildcard(ptl))
 		return &mtable->mt_mhash[mbits & LNET_MT_HASH_MASK];
-	} else {
-		unsigned long hash = mbits + id.nid + id.pid;
-
-		LASSERT(lnet_ptl_is_unique(ptl));
-		hash = hash_long(hash, LNET_MT_HASH_BITS);
-		return &mtable->mt_mhash[hash];
-	}
+	hash = mbits + id.nid + id.pid;
+		
+	LASSERT(lnet_ptl_is_unique(ptl));
+	hash = hash_long(hash, LNET_MT_HASH_BITS);
+	return &mtable->mt_mhash[hash];
 }
 
 int
-- 
1.9.1