openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

staging: wilc1000: removes redundant 'continue' in while loop conditional blocks 

Removing 6 continue statements from a while loop. The
continue statements are redundant here since control
already returns to the beginning of the loop upon exit
of any of the conditional blocks. Found using Coccinelle.

Signed-off-by: Tahia Khan <tahia.khan@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Tahia Khan 2017-03-08 18:21:38 -05:00 committed by Greg Kroah-Hartman
parent 28c8285955
commit 069baa6a5e
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/staging/wilc1000/host_interface.c
View File

@ -3884,7 +3884,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
pNewJoinBssParam->supp_rates[i + 1] = pu8IEs[index + i]; pNewJoinBssParam->supp_rates[i + 1] = pu8IEs[index + i];
index += suppRatesNo; index += suppRatesNo;
continue;
} else if (pu8IEs[index] == EXT_SUPP_RATES_IE) { } else if (pu8IEs[index] == EXT_SUPP_RATES_IE) {
extSuppRatesNo = pu8IEs[index + 1]; extSuppRatesNo = pu8IEs[index + 1];
if (extSuppRatesNo > (MAX_RATES_SUPPORTED - suppRatesNo)) if (extSuppRatesNo > (MAX_RATES_SUPPORTED - suppRatesNo))
@ -3896,11 +3895,9 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
pNewJoinBssParam->supp_rates[suppRatesNo + i + 1] = pu8IEs[index + i]; pNewJoinBssParam->supp_rates[suppRatesNo + i + 1] = pu8IEs[index + i];
index += extSuppRatesNo; index += extSuppRatesNo;
continue;
} else if (pu8IEs[index] == HT_CAPABILITY_IE) { } else if (pu8IEs[index] == HT_CAPABILITY_IE) {
pNewJoinBssParam->ht_capable = true; pNewJoinBssParam->ht_capable = true;
index += pu8IEs[index + 1] + 2; index += pu8IEs[index + 1] + 2;
continue;
} else if ((pu8IEs[index] == WMM_IE) && } else if ((pu8IEs[index] == WMM_IE) &&
(pu8IEs[index + 2] == 0x00) && (pu8IEs[index + 3] == 0x50) && (pu8IEs[index + 2] == 0x00) && (pu8IEs[index + 3] == 0x50) &&
(pu8IEs[index + 4] == 0xF2) && (pu8IEs[index + 4] == 0xF2) &&
@ -3912,7 +3909,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
if (pu8IEs[index + 8] & BIT(7)) if (pu8IEs[index + 8] & BIT(7))
pNewJoinBssParam->uapsd_cap = true; pNewJoinBssParam->uapsd_cap = true;
index += pu8IEs[index + 1] + 2; index += pu8IEs[index + 1] + 2;
continue;
} else if ((pu8IEs[index] == P2P_IE) && } else if ((pu8IEs[index] == P2P_IE) &&
(pu8IEs[index + 2] == 0x50) && (pu8IEs[index + 3] == 0x6f) && (pu8IEs[index + 2] == 0x50) && (pu8IEs[index + 3] == 0x6f) &&
(pu8IEs[index + 4] == 0x9a) && (pu8IEs[index + 4] == 0x9a) &&
@ -3942,8 +3938,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
memcpy(pNewJoinBssParam->start_time, pu8IEs + u16P2P_count, 4); memcpy(pNewJoinBssParam->start_time, pu8IEs + u16P2P_count, 4);
index += pu8IEs[index + 1] + 2; index += pu8IEs[index + 1] + 2;
continue;
} else if ((pu8IEs[index] == RSN_IE) || } else if ((pu8IEs[index] == RSN_IE) ||
((pu8IEs[index] == WPA_IE) && (pu8IEs[index + 2] == 0x00) && ((pu8IEs[index] == WPA_IE) && (pu8IEs[index + 2] == 0x00) &&
(pu8IEs[index + 3] == 0x50) && (pu8IEs[index + 4] == 0xF2) && (pu8IEs[index + 3] == 0x50) && (pu8IEs[index + 4] == 0xF2) &&
@ -3989,7 +3983,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
} }
pNewJoinBssParam->rsn_found = true; pNewJoinBssParam->rsn_found = true;
index += pu8IEs[index + 1] + 2; index += pu8IEs[index + 1] + 2;
continue;
} else { } else {
index += pu8IEs[index + 1] + 2; index += pu8IEs[index + 1] + 2;
} }