Thread (3 messages) 3 messages, 3 authors, 2021-10-25

Re: [PATCH] staging: rtl8723bs: core: Refactor nested if-else

From: Greg KH <gregkh@linuxfoundation.org>
Date: 2021-10-25 07:33:22
Also in: lkml

On Mon, Oct 25, 2021 at 12:55:28PM +0530, Kushal Kothari wrote:
Refactor nested if else by combining nested if into a single if condition and removing unnecessary else conditionals which leads to removing unnecessary tabs .There is no change in logic of new code.
Very long line, please break it up at 72 columns.

And your space around the '.' is odd :(
checkpatch warning : Too many leading tabs - consider code refactoring
What does this mean?
Signed-off-by: Kushal Kothari <redacted>
---
 drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 65 ++++++++-----------
 1 file changed, 26 insertions(+), 39 deletions(-)
thanks,

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help