Re: [PATCH] staging: rtl8712: Fix unnecessary parentheses warnings

From: Xingquan Liu
Date: Fri Sep 27 2024 - 11:53:56 EST


On Fri Sep 27, 2024 at 11:45 PM CST, Dan Carpenter wrote:
Greg doesn't agree with checkpatch about unnecessary parentheses. If the code
looks like this:

if ((a == b) || (b == c)) {

then just ignore checkpatch. However if the code looks like:

p = &(foo);

Then change it to:

p = &foo;

This sort of change would need to be split up by driver instead of all of
driver/staging. So it would be a series of patches. You don't need to split
it up by file. Split it by driver.

Thanks, this information is important.

--
Xingquan Liu

Attachment: signature.asc
Description: PGP signature