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


Groups > linux.kernel > #1225133 > unrolled thread

[PATCH] staging: wilc1000: Removed curly braces

Started byAparna Karuthodi <kdasaparna@gmail.com>
First post2015-09-15 16:00 +0200
Last post2015-09-16 06:40 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] staging: wilc1000: Removed curly braces Aparna Karuthodi <kdasaparna@gmail.com> - 2015-09-15 16:00 +0200
    Re: [PATCH] staging: wilc1000: Removed curly braces Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-16 06:40 +0200

#1225133 — [PATCH] staging: wilc1000: Removed curly braces

FromAparna Karuthodi <kdasaparna@gmail.com>
Date2015-09-15 16:00 +0200
Subject[PATCH] staging: wilc1000: Removed curly braces
Message-ID<q8YW6-6Co-27@gated-at.bofh.it>
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

--
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/

[toc] | [next] | [standalone]


#1225712

FromSudip Mukherjee <sudipm.mukherjee@gmail.com>
Date2015-09-16 06:40 +0200
Message-ID<q9cFH-2eH-9@gated-at.bofh.it>
In reply to#1225133
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
--
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/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web