Messages in this thread | | | From | Miguel Ojeda <> | Date | Tue, 12 Apr 2022 19:35:43 +0200 | Subject | Re: [PATCH v1] clang-format: Update and extend the for_each list with tools/ |
| |
On Tue, Apr 12, 2022 at 7:05 PM Mickaël Salaün <mic@digikod.net> wrote: > > I'm wondering about the ASSERT_* and EXPECT_* macros from > tools/testing/selftests/kselftest_harness.h > Do you think we should treat them as "for macros" as well? They can > either be used with or without a following code block.
I would keep the style that we have in the kernel -- as far as I can see, that means without a space.
Also, from a quick grep, it seems most callers do not use the optional handler, which also makes sense w.r.t. not using a space.
Cheers, Miguel
| |