Messages in this thread | | | Date | Sat, 20 Nov 2021 17:44:31 +0100 | Subject | Re: [PATCH v2 11/20] linux/container_of.h: Cosmetic | From | "Alejandro Colomar (man-pages)" <> |
| |
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@gmail.com> 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/
| |