Re: [PATCH] staging: rtl8712: aggregate conditional statements

From: Dan Carpenter
Date: Mon Mar 18 2024 - 03:47:01 EST


On Sun, Mar 17, 2024 at 11:36:17PM +0530, Ayush Tiwari wrote:
> Aggregate the two conditional statements into one conditional
> statement to reduce code length and simplify the code structure.
>
> Signed-off-by: Ayush Tiwari <ayushtiw0110@xxxxxxxxx>

I'm not a fan of this sort of thing. The new code is fine but the
original code was fine. If you were going to be taking over the
maintainership of the rtl8712 driver, then you could do whatever you
want. But if you're just sending drive by patches, then it's just leave
the original code as is. You might be right. But it's not worth
debating or even thinking about for more than a second or two.

Someone was complaining about my code recently where I tend to break
things up into separate if statements and they were like "You could
combine them all on one line", and I'm like, "Yeah. It was that way to
begin with but I was fixing some issues and I deliberately separated it
out for readability." So that's one life hack which that if you don't
like someone's code style, just find some mistakes and re-write it.

regards,
dan carpenter