On Monday, October 25, 2021 9:56:00 AM CEST kushal kothari wrote: > >Very long line, please break it up at 72 columns. > > >And your space around the '.' is odd :( > > Yes,fixing it. > ^^^^^^^^^^^^^^^^^^^^ You are still using spaces the wrong way :) English, as well as all natural languages that use Latin characters, use punctuation and a strict style that dictates how to use the spaces around it. Please read correctly written English texts, learn and use conventions Thanks, Fabio > > >> checkpatch warning : Too many leading tabs - consider code refactoring > > >What does this mean? > Asking about the warning ,correct? > The current code is using very deep indentations (which can be avoided) and > due to this there are many leading tabs. So the checkpatch.pl warns of "Too > many leading tabs - consider code refactoring" > <http://checkpatch.pl> > > On Mon, Oct 25, 2021 at 1:03 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > 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 <kushalkothari285@xxxxxxxxx> > > > --- > > > drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 65 ++++++++----------- > > > 1 file changed, 26 insertions(+), 39 deletions(-) > > > > > > > thanks, > > > > greg k-h > > >