Staging: rtl8188eu: Remove redundant local variable
authorSomya Anand <somyaanand214@gmail.com>
Wed, 4 Mar 2015 08:40:44 +0000 (14:10 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Mar 2015 23:30:07 +0000 (15:30 -0800)
This patch removes a redundant variable "raid" and adds inline
return statements.

This issue is identified by the following coccinelle script:
@@
expression ret;
identifier f;
@@

-ret =
+return
     f(...);
-return ret;

Signed-off-by: Somya Anand <somyaanand214@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_wlan_util.c

index a3ffc691be9a24396e9967750916e0af5ee03a2b..6917abf21ec04fbfc7343cf5f6c77b875461a95f 100644 (file)
@@ -88,35 +88,25 @@ int cckratesonly_included(unsigned char *rate, int ratelen)
 
 unsigned char networktype_to_raid(unsigned char network_type)
 {
-       unsigned char raid;
-
        switch (network_type) {
        case WIRELESS_11B:
-               raid = RATR_INX_WIRELESS_B;
-               break;
+               return RATR_INX_WIRELESS_B;
        case WIRELESS_11A:
        case WIRELESS_11G:
-               raid = RATR_INX_WIRELESS_G;
-               break;
+               return RATR_INX_WIRELESS_G;
        case WIRELESS_11BG:
-               raid = RATR_INX_WIRELESS_GB;
-               break;
+               return RATR_INX_WIRELESS_GB;
        case WIRELESS_11_24N:
        case WIRELESS_11_5N:
-               raid = RATR_INX_WIRELESS_N;
-               break;
+               return RATR_INX_WIRELESS_N;
        case WIRELESS_11A_5N:
        case WIRELESS_11G_24N:
-               raid = RATR_INX_WIRELESS_NG;
-               break;
+               return  RATR_INX_WIRELESS_NG;
        case WIRELESS_11BG_24N:
-               raid = RATR_INX_WIRELESS_NGB;
-               break;
+               return RATR_INX_WIRELESS_NGB;
        default:
-               raid = RATR_INX_WIRELESS_GB;
-               break;
+               return RATR_INX_WIRELESS_GB;
        }
-       return raid;
 }
 
 u8 judge_network_type(struct adapter *padapter, unsigned char *rate, int ratelen)