Re: [PATCH v3] Documentation/process/coding-style.rst: space around const

From: Miguel Ojeda
Date: Thu Oct 12 2023 - 12:54:00 EST


On Thu, Oct 12, 2023 at 4:48 PM Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> Do older versions of clang-format ignore entries
> they don't understand?

Sadly, no, that is the reason we keep it at the minimum.

However, I just took a look again at it, and I see that such support
was added to LLVM 12, the `--Wno-error=unknown` flag in commit
f64903fd8176 ("Add -Wno-error=unknown flag to clang-format.").

So this means that the minimum is bumped to 12, we could in principle
use newer options.

I think the downsides are that users will need to pass the flag
(potentially in e.g. their IDE or similar) and that formatting could
be potentially chaotic depending on the options ignored. I guess
particular subsystems could agree on which version to use.

Cheers,
Miguel