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

From: Joe Perches
Date: Sat Nov 20 2021 - 11:33:28 EST


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 {