Messages in this thread Patch in this message | | | Subject | [RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING (checkpatch) | From | Sohom Datta <> | Date | Sun, 13 Sep 2020 12:19:50 +0530 |
| |
From 4c8c8f3ff7f4d711daea4ac3bb987fcecc7ef1ed Mon Sep 17 00:00:00 2001 From: Sohom <sohom.datta@learner.manipal.edu> Date: Sat, 12 Sep 2020 18:04:56 +0530 Subject: [RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING (checkpatch)
Fixed: WARNING: else is not generally useful after a break or return 1636: FILE: ./rtw_recv.c:1636: + return false; + else
Separated the return statement into a separate block since it doesn't seem to depend on the SN_LESS explicity being false.
Signed-off-by: Sohom <sohom.datta@learner.manipal.edu> --- drivers/staging/rtl8188eu/core/rtw_recv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c b/drivers/staging/rtl8188eu/core/rtw_recv.c index 5fe7a0458dd2..5e81134ffb6d 100644 --- a/drivers/staging/rtl8188eu/core/rtw_recv.c +++ b/drivers/staging/rtl8188eu/core/rtw_recv.c @@ -1629,10 +1629,11 @@ static int enqueue_reorder_recvframe(struct recv_reorder_ctrl *preorder_ctrl, hdr = list_entry(plist, struct recv_frame, list); pnextattrib = &hdr->attrib; + if (SN_EQUAL(pnextattrib->seq_num, pattrib->seq_num)) + return false; + if (SN_LESS(pnextattrib->seq_num, pattrib->seq_num)) plist = plist->next; - else if (SN_EQUAL(pnextattrib->seq_num, pattrib->seq_num)) - return false; else break; } -- 2.28.0
| |