Messages in this thread | | | Date | Wed, 13 Apr 2022 11:52:59 +0300 | Subject | Re: [PATCH] staging: r8188eu: remove else after return and break statements | From | Pavel Skripkin <> |
| |
Hi Mahak,
On 4/13/22 08:27, Mahak Gupta wrote: > Else is not necessary after return and break statements, hence remove > it. > > Reported by checkpatch: > > WARNING: else is not generally useful after a break or return > > Signed-off-by: Mahak Gupta <mahak_g@cs.iitr.ac.in>
[snip]
> - } > + /* could be pure B, pure G, or B/G */ > + if (rtw_is_cckratesonly_included(rate)) > + return WIRELESS_11B; > + else if (rtw_is_cckrates_included(rate)) > + return WIRELESS_11BG; > + else > + return WIRELESS_11G;
This 'else' is after 'return' as well, isn't it? Just wondering what's the difference between this one and others in this patch
With regards, Pavel Skripkin
| |