staging: rtl8192u: Removed assignments from if statements.
authorChi Pham <fempsci@gmail.com>
Mon, 10 Mar 2014 21:31:52 +0000 (22:31 +0100)
committerPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Tue, 11 Mar 2014 05:57:11 +0000 (22:57 -0700)
Removes assignments from if statements and simplifies unnecessary
0/NULL-checking. The following coccinelle script found the match:
@@
expression E0, E1, E2;
statement S0;
@@
- if (E0 == (E1 = E2))
+ E1 = E2;
+ if (E1 == E0)
S0

Signed-off-by: Chi Pham <fempsci@gmail.com>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
drivers/staging/rtl8192u/r8192U_core.c

index 662c7e41cd5cec78a9ade7c7ae1703c7db01c262..9d4bc7b7f71f4313bb0a3f0c984cc810d26928fa 100644 (file)
@@ -1959,7 +1959,8 @@ ieee80211_rx_frame_softmac(struct ieee80211_device *ieee, struct sk_buff *skb,
                        struct ieee80211_network network_resp;
                        struct ieee80211_network *network = &network_resp;
 
-                       if (0 == (errcode=assoc_parse(ieee,skb, &aid))){
+                       errcode = assoc_parse(ieee, skb, &aid);
+                       if (!errcode) {
                                ieee->state=IEEE80211_LINKED;
                                ieee->assoc_id = aid;
                                ieee->softmac_stats.rx_ass_ok++;
@@ -2017,7 +2018,8 @@ ieee80211_rx_frame_softmac(struct ieee80211_device *ieee, struct sk_buff *skb,
 
                                        IEEE80211_DEBUG_MGMT("Received authentication response");
 
-                                       if (0 == (errcode=auth_parse(skb, &challenge, &chlen))){
+                                       errcode = auth_parse(skb, &challenge, &chlen);
+                                       if (!errcode) {
                                                if(ieee->open_wep || !challenge){
                                                        ieee->state = IEEE80211_ASSOCIATING_AUTHENTICATED;
                                                        ieee->softmac_stats.rx_auth_rs_ok++;
index a56be798c164ef841bc55522cf6759346df04989..834afca0c74bc5ab828a276901d4859e764b9cbf 100644 (file)
@@ -1318,7 +1318,8 @@ static void rtl8192_tx_isr(struct urb *tx_urb)
                /* Don't send data frame during scanning.*/
                if ((skb_queue_len(&priv->ieee80211->skb_waitQ[queue_index]) != 0) &&
                    (!(priv->ieee80211->queue_stop))) {
-                       if (NULL != (skb = skb_dequeue(&(priv->ieee80211->skb_waitQ[queue_index]))))
+                       skb = skb_dequeue(&(priv->ieee80211->skb_waitQ[queue_index]));
+                       if (skb)
                                priv->ieee80211->softmac_hard_start_xmit(skb, dev);
 
                        return; //modified by david to avoid further processing AMSDU