Re: [PATCH v2] staging: rtl8712: Adjust if statements to reduce indentation level

2018-06-16 Thread Joe Perches
On Sat, 2018-06-16 at 15:03 +0900, Joonhwan Kim wrote: > Merge two condition of if statements in > r8712_surveydone_event_callback() Are you sure you are not changing the logic here? I think it'd be nicer to refactor the code instead. Something like: --- drivers/staging/rtl8712/rtl871x_mlme.c |

[PATCH v2] staging: rtl8712: Adjust if statements to reduce indentation level

2018-06-15 Thread Joonhwan Kim
Merge two condition of if statements in r8712_surveydone_event_callback() Signed-off-by: Joonhwan Kim --- v2: fix my name without period in email header drivers/staging/rtl8712/rtl871x_mlme.c | 51 +- 1 file changed, 25 insertions(+), 26 deletions(-) diff --git a/driver