Re: [PATCH] net: bridge: Slightly optimize 'find_portno()'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le 15/11/2021 à 13:35, Dan Carpenter a écrit :
On Sun, Nov 14, 2021 at 08:02:35PM +0100, Christophe JAILLET wrote:
The 'inuse' bitmap is local to this function. So we can use the
non-atomic '__set_bit()' to save a few cycles.

While at it, also remove some useless {}.

I like the {} and tend to add it in new code.  There isn't a rule about
this one way or the other.

regards,
dan carpenter




Hi Dan,

- checkpatch prefers the style without {}
- Usually, greg k-h and Joe Perches give feed-back that extra {} should be removed. - in https://www.kernel.org/doc/html/v5.13/process/coding-style.html, after "Rationale: K&R":
   "Do not unnecessarily use braces where a single statement will do."


My own preference is to have {}. It is the standard used on another project I work on (i.e. httpd) and it helps when you add some code (it avoids unexpected behavior if you forget to add some missing {})

My understanding is that on the HUGE code base of Linux, emphasis is put on saving some lines of code, reducing the length of lines and avoiding the need to read some extra char.
I'm also fine with it.

CJ



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux