Message-ID: <8d99e5cc64db64b6262b74aa7a9e04a642a7629f.camel@perches.com>
Subject: Re: [PATCH v2 11/20] linux/container_of.h: Cosmetic
From:   Joe Perches <joe@perches.com>
To:     Andy Shevchenko <andy.shevchenko@gmail.com>,
        Alejandro Colomar <alx.manpages@gmail.com>
Cc:     LKML <linux-kernel@vger.kernel.org>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Arnd Bergmann <arnd@arndb.de>,
        Alexey Dobriyan <adobriyan@gmail.com>,
        Jani Nikula <jani.nikula@linux.intel.com>,
        Rasmus Villemoes <linux@rasmusvillemoes.dk>,
        Kees Cook <keescook@chromium.org>
Date:   Sat, 20 Nov 2021 08:33:21 -0800
In-Reply-To: <CAHp75VfYvO2pHTtExLpcY1AmiGAEvvdf_Q_46ZhxR6JD68MNow@mail.gmail.com>
References: <20211119113644.1600-1-alx.manpages@gmail.com>
         <20211120130104.185699-1-alx.manpages@gmail.com>
         <20211120130104.185699-12-alx.manpages@gmail.com>
         <CAHp75VfYvO2pHTtExLpcY1AmiGAEvvdf_Q_46ZhxR6JD68MNow@mail.gmail.com>
Content-Type: text/plain; charset="ISO-8859-1"
User-Agent: Evolution 3.40.4-1 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk

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 		{