Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1219542

[PATCH 04/10] Staging: lustre: obdclass: Remove braces for single statement blocks

From Shraddha Barke <shraddha.6596@gmail.com>
Newsgroups linux.kernel
Subject [PATCH 04/10] Staging: lustre: obdclass: Remove braces for single statement blocks
Date 2015-09-05 15:40 +0200
Message-ID <q5lRh-1T7-19@gated-at.bofh.it> (permalink)
References <q5lHz-1Hk-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


The linux kernel coding style discourages use of braces for single
statement blocks. This patch removes the unnecessary braces.

The warning was detected using checkpatch.pl. Coccinelle was used to
make the change.

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
 drivers/staging/lustre/lustre/obdclass/dt_object.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/lustre/lustre/obdclass/dt_object.c b/drivers/staging/lustre/lustre/obdclass/dt_object.c
index b67b0fe..6b645a1 100644
--- a/drivers/staging/lustre/lustre/obdclass/dt_object.c
+++ b/drivers/staging/lustre/lustre/obdclass/dt_object.c
@@ -778,7 +778,7 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
 		return PTR_ERR(it);
 
 	rc = iops->load(env, it, rdpg->rp_hash);
-	if (rc == 0) {
+	if (rc == 0)
 		/*
 		 * Iterator didn't find record with exactly the key requested.
 		 *
@@ -790,9 +790,8 @@ int dt_index_walk(const struct lu_env *env, struct dt_object *obj,
 		 *     state)---position it on the next item.
 		 */
 		rc = iops->next(env, it);
-	} else if (rc > 0) {
+	else if (rc > 0)
 		rc = 0;
-	}
 
 	/* Fill containers one after the other. There might be multiple
 	 * containers per physical page.
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 01/10] Staging: rtl8723au: hal: Remove braces for single statement blocks Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:30 +0200
  [PATCH 06/10] Staging: lustre: llite: llite_lib.c: Remove unnecessary braces Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:30 +0200
  [PATCH 10/10] Staging: rtl8192e: rtl_wx.c: Remove braces for single statement blocks Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:30 +0200
  [PATCH 09/10] Staging: speakup: Remove braces for single statement blocks Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:30 +0200
  [PATCH 04/10] Staging: lustre: obdclass: Remove braces for single statement blocks Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:40 +0200
  [PATCH 05/10] Staging: lustre: llite: llite_close.c: Remove unnecessary braces Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:40 +0200
  [PATCH 08/10] Staging: android: Remove unnecessary braces. Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:40 +0200
  [PATCH 02/10] Staging: lustre: ptlrpc: Remove braces for single statement blocks Shraddha Barke <shraddha.6596@gmail.com> - 2015-09-05 15:40 +0200

csiph-web