Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 86c0205b authored by Quytelda Kahja's avatar Quytelda Kahja Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: Remove braces from single statement conditionals. 



Several conditionals in rtw_wx_set_auth() contain a comment then a
single statement.  Move the comments to the top of the conditionals
so that braces can be removed from the statements, which saves space
and makes the code more readable.

Signed-off-by: default avatarQuytelda Kahja <quytelda@tamalin.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 41ed8df9

drivers/staging/rtl8723bs/os_dep/ioctl_linux.c

+9 −9
Original line number Diff line number Diff line
@@ -2151,13 +2151,11 @@ static int rtw_wx_set_auth(struct net_device *dev, 
		 */

		break;

	case IW_AUTH_TKIP_COUNTERMEASURES:

		if (param->value) {

			/*  wpa_supplicant is enabling the tkip countermeasure. */

		/* wpa_supplicant is setting the tkip countermeasure. */

		if (param->value) /* enabling */

			padapter->securitypriv.btkip_countermeasure = true;

		} else {

			/*  wpa_supplicant is disabling the tkip countermeasure. */

		else /* disabling */

			padapter->securitypriv.btkip_countermeasure = false;

		}

		break;

	case IW_AUTH_DROP_UNENCRYPTED:

		/* HACK:
@@ -2172,10 +2170,12 @@ static int rtw_wx_set_auth(struct net_device *dev, 
		 * be set.

		 */



		if (padapter->securitypriv.ndisencryptstatus == Ndis802_11Encryption1Enabled) {

			break;/* it means init value, or using wep, ndisencryptstatus = Ndis802_11Encryption1Enabled, */

			/*  then it needn't reset it; */

		}

		/*

		 * This means init value, or using wep, ndisencryptstatus =

		 * Ndis802_11Encryption1Enabled, then it needn't reset it;

		 */

		if (padapter->securitypriv.ndisencryptstatus == Ndis802_11Encryption1Enabled)

			break;



		if (param->value) {

			padapter->securitypriv.ndisencryptstatus = Ndis802_11EncryptionDisabled;