Re: [PATCH v2 11/20] linux/container_of.h: Cosmetic

From: Alejandro Colomar (man-pages)
Date: Sat Nov 20 2021 - 11:44:36 EST


Hi Joe and Andy,

On 11/20/21 17:33, Joe Perches wrote:
On Sat, 2021-11-20 at 18:12 +0200, Andy Shevchenko wrote:
On Sat, Nov 20, 2021 at 3:03 PM Alejandro Colomar
<alx.manpages@xxxxxxxxx> wrote:

Place braces in a ({}) expression
similarly to how a function would have them.

I don't think we need this change. ({ on one line is pretty much
idiomatic for macros.

true

$ grep-2.5.4 -rP --include=*.[ch] -oh '\(\s*\\?\s*\{' * | \
sort | uniq -c | sort -rn
2756 ({
13 {
13 ( {
4 ( \
2 ( \
2 ( \
2 ( \
1 ( \
1 ( \
1 ( \
1 ( \
1 {



Thanks,
I'll drop that one.

Cheers,
Alex

--
Alejandro Colomar
Linux man-pages comaintainer; http://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/