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


Groups > linux.kernel > #1219543

[PATCH 05/10] Staging: lustre: llite: llite_close.c: Remove unnecessary braces

From Shraddha Barke <shraddha.6596@gmail.com>
Newsgroups linux.kernel
Subject [PATCH 05/10] Staging: lustre: llite: llite_close.c: Remove unnecessary braces
Date 2015-09-05 15:40 +0200
Message-ID <q5lRi-1T7-37@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/llite/llite_close.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/lustre/lustre/llite/llite_close.c b/drivers/staging/lustre/lustre/llite/llite_close.c
index 7bdae72..e31cad8 100644
--- a/drivers/staging/lustre/lustre/llite/llite_close.c
+++ b/drivers/staging/lustre/lustre/llite/llite_close.c
@@ -293,14 +293,13 @@ static void ll_done_writing(struct inode *inode)
 		goto out;
 
 	rc = md_done_writing(ll_i2sbi(inode)->ll_md_exp, op_data, NULL);
-	if (rc == -EAGAIN) {
+	if (rc == -EAGAIN)
 		/* MDS has instructed us to obtain Size-on-MDS attribute from
 		 * OSTs and send setattr to back to MDS. */
 		rc = ll_som_update(inode, op_data);
-	} else if (rc) {
+	else if (rc)
 		CERROR("inode %lu mdc done_writing failed: rc = %d\n",
 		       inode->i_ino, rc);
-	}
 out:
 	ll_finish_md_op_data(op_data);
 	if (och) {
-- 
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