staging: wilc1000: removes redundant 'continue' in while loop conditional blocks
authorTahia Khan <tahia.khan@gmail.com>
Wed, 8 Mar 2017 23:21:38 +0000 (18:21 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 9 Mar 2017 13:55:09 +0000 (14:55 +0100)
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>
drivers/staging/wilc1000/host_interface.c

index 0ffa4896d5da6a39ef87b8e7a928db70dbf2b283..7a9e8d3c4a367d77584f3043c5479ad507ac00e3 100644 (file)
@@ -3884,7 +3884,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
                                        pNewJoinBssParam->supp_rates[i + 1] = pu8IEs[index + i];
 
                                index += suppRatesNo;
-                               continue;
                        } else if (pu8IEs[index] == EXT_SUPP_RATES_IE) {
                                extSuppRatesNo = pu8IEs[index + 1];
                                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];
 
                                index += extSuppRatesNo;
-                               continue;
                        } else if (pu8IEs[index] == HT_CAPABILITY_IE) {
                                pNewJoinBssParam->ht_capable = true;
                                index += pu8IEs[index + 1] + 2;
-                               continue;
                        } else if ((pu8IEs[index] == WMM_IE) &&
                                   (pu8IEs[index + 2] == 0x00) && (pu8IEs[index + 3] == 0x50) &&
                                   (pu8IEs[index + 4] == 0xF2) &&
@@ -3912,7 +3909,6 @@ static void *host_int_ParseJoinBssParam(struct network_info *ptstrNetworkInfo)
                                if (pu8IEs[index + 8] & BIT(7))
                                        pNewJoinBssParam->uapsd_cap = true;
                                index += pu8IEs[index + 1] + 2;
-                               continue;
                        } else if ((pu8IEs[index] == P2P_IE) &&
                                 (pu8IEs[index + 2] == 0x50) && (pu8IEs[index + 3] == 0x6f) &&
                                 (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);
 
                                index += pu8IEs[index + 1] + 2;
-                               continue;
-
                        } else if ((pu8IEs[index] == RSN_IE) ||
                                 ((pu8IEs[index] == WPA_IE) && (pu8IEs[index + 2] == 0x00) &&
                                  (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;
                                index += pu8IEs[index + 1] + 2;
-                               continue;
                        } else {
                                index += pu8IEs[index + 1] + 2;
                        }