linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] staging: wilc1000: Removed curly braces
@ 2015-09-15 13:54 Aparna Karuthodi
  2015-09-16  4:35 ` Sudip Mukherjee
  0 siblings, 1 reply; 2+ messages in thread
From: Aparna Karuthodi @ 2015-09-15 13:54 UTC (permalink / raw)
  To: gregkh
  Cc: linux-wireless, devel, linux-kernel, johnny.kim, rachel.kim,
	Aparna Karuthodi

Removed the curly braces of a single statement if block to remove a
coding style warning detected by checkpatch.
The warning is given below:
WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Aparna Karuthodi <kdasaparna@gmail.com>
---
 drivers/staging/wilc1000/coreconfigurator.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c
index 0c964c6..5241699 100644
--- a/drivers/staging/wilc1000/coreconfigurator.c
+++ b/drivers/staging/wilc1000/coreconfigurator.c
@@ -1944,9 +1944,8 @@ s32 SendConfigPkt(u8 u8Mode, tstrWID *pstrWIDs,
 	gstrConfigPktInfo.bRespRequired = bRespRequired;
 
 	s32Error = SendRawPacket(gps8ConfigPacket, s32ConfigPacketLen);
-	if (s32Error != WILC_SUCCESS) {
+	if (s32Error != WILC_SUCCESS)
 		goto End_ConfigPkt;
-	}
 
 	WILC_memset((void *)gps8ConfigPacket, 0, MAX_PACKET_BUFF_SIZE);
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] staging: wilc1000: Removed curly braces
  2015-09-15 13:54 [PATCH] staging: wilc1000: Removed curly braces Aparna Karuthodi
@ 2015-09-16  4:35 ` Sudip Mukherjee
  0 siblings, 0 replies; 2+ messages in thread
From: Sudip Mukherjee @ 2015-09-16  4:35 UTC (permalink / raw)
  To: Aparna Karuthodi
  Cc: gregkh, devel, rachel.kim, linux-wireless, linux-kernel, johnny.kim

On Tue, Sep 15, 2015 at 07:24:00PM +0530, Aparna Karuthodi wrote:
> Removed the curly braces of a single statement if block to remove a
> coding style warning detected by checkpatch.
> The warning is given below:
> WARNING: braces {} are not necessary for single statement blocks
> 
> Signed-off-by: Aparna Karuthodi <kdasaparna@gmail.com>
> ---
This will not apply any more due to recent modification to this file.

regards
sudip

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-16  4:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-15 13:54 [PATCH] staging: wilc1000: Removed curly braces Aparna Karuthodi
2015-09-16  4:35 ` Sudip Mukherjee

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).