Re: [PATCH v2 4/5] clang-format: Fix empty curly braces

From: Joe Perches
Date: Fri May 20 2022 - 14:00:13 EST


On Fri, 2022-05-20 at 19:50 +0200, Miguel Ojeda wrote:
> Hi Joe,
>
> On Fri, May 20, 2022 at 7:24 PM Joe Perches <joe@xxxxxxxxxxx> wrote:
> >
> > static inline void foo1(...) {}
> > static inline void foo2(...) {}
> > ...
> > static inline void fooN(...) {}
>
> Those are the non-wrapped-from-signature case, which are handled
> before this option takes place, i.e. this option is about:
>
> void f()
> {}
>
> vs.
>
> void f()
> {
> }
>
> For putting everything on the same line, we could adjust
> `AllowShortFunctionsOnASingleLine` from `None` to `Empty`, though.

As you wish.

I think it's more that there's no "one true style" issue than anything.