2015-09-15 13:52:03

by Aparna Karuthodi

Subject: [PATCH] staging: wilc1000: Removed curly braces

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 <[email protected]>
---
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

2015-09-16 04:35:52

by Sudip Mukherjee

Subject: Re: [PATCH] staging: wilc1000: Removed curly braces

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 <[email protected]>
> ---
This will not apply any more due to recent modification to this file.

regards
sudip