Messages in this thread | | | From | Miguel Ojeda <> | Date | Fri, 20 May 2022 19:15:11 +0200 | Subject | Re: [PATCH v2 4/5] clang-format: Fix empty curly braces |
| |
On Fri, May 6, 2022 at 6:00 PM Mickaël Salaün <mic@digikod.net> wrote: > > SplitEmptyFunction [1] should be false to not add a new line in an empty > function body. This follows the current kernel coding style.
I don't think this is correct. Checking headers in `include/linux/`, I see ~70 using {} (when not in the same line as the signature), and ~700 in different lines. In `kernel/`, it is even more pronounced: 4 vs. ~200.
Am I missing something?
Cheers, Miguel
| |