OSDN Git Service

staging: rtl8188eu: core: Remove unnecessary braces
authorBhaktipriya Shridhar <bhaktipriya96@gmail.com>
Thu, 18 Feb 2016 22:48:24 +0000 (04:18 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Feb 2016 23:06:51 +0000 (15:06 -0800)
This patch removes braces for single statement blocks. The warning
was detected using checkpatch.pl.
Coccinelle was used to make the change.

@@
expression e,e1;
@@

- if (e) {
+ if (e)
  e1;
- }

Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_recv.c

index c81639c..40b7a30 100644 (file)
@@ -116,9 +116,8 @@ void _rtw_free_recv_priv(struct recv_priv *precvpriv)
 
        rtw_free_uc_swdec_pending_queue(padapter);
 
-       if (precvpriv->pallocated_frame_buf) {
+       if (precvpriv->pallocated_frame_buf)
                vfree(precvpriv->pallocated_frame_buf);
-       }
 
        rtw_hal_free_recv_priv(padapter);
 
@@ -910,9 +909,8 @@ static int sta2ap_data_frame(struct adapter *adapter,
 
                process_pwrbit_data(adapter, precv_frame);
 
-               if ((GetFrameSubType(ptr) & WIFI_QOS_DATA_TYPE) == WIFI_QOS_DATA_TYPE) {
+               if ((GetFrameSubType(ptr) & WIFI_QOS_DATA_TYPE) == WIFI_QOS_DATA_TYPE)
                        process_wmmps_data(adapter, precv_frame);
-               }
 
                if (GetFrameSubType(ptr) & BIT(6)) {
                        /* No data, will not indicate to upper layer, temporily count it here */
@@ -1527,10 +1525,9 @@ struct recv_frame *recvframe_chk_defrag(struct adapter *padapter,
                if (pdefrag_q != NULL) {
                        if (fragnum == 0) {
                                /* the first fragment */
-                               if (!list_empty(&pdefrag_q->queue)) {
+                               if (!list_empty(&pdefrag_q->queue))
                                        /* free current defrag_q */
                                        rtw_free_recvframe_queue(pdefrag_q, pfree_recv_queue);
-                               }
                        }
 
                        /* Then enqueue the 0~(n-1) fragment into the defrag_q */
@@ -1646,9 +1643,8 @@ static int amsdu_to_msdu(struct adapter *padapter, struct recv_frame *prframe)
                a_len -= nSubframe_Length;
                if (a_len != 0) {
                        padding_len = 4 - ((nSubframe_Length + ETH_HLEN) & (4-1));
-                       if (padding_len == 4) {
+                       if (padding_len == 4)
                                padding_len = 0;
-                       }
 
                        if (a_len < padding_len) {
                                goto exit;