[clang-tools-extra] [clang-tidy][NFC] add qutation mark for C++ classes in warning message (PR #109068)
Congcong Cai via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 18 04:42:43 PDT 2024
HerrCai0907 wrote:
> LGTM, thanks for splitting the patch!
>
> Personally I don't quite see the added value of having the quotes, I actually find them a bit noisy. It would have been good to write the motivation for this change in the commit message.
>
> But this is quite subjective so if you find that it makes the diagnostic clearer then I won't object.
I think it is more readable to use 'std::array' than std::array<>. The quotes at least is a highlight mark / the wrapper of code. But the template mark is really hard to understand.
https://github.com/llvm/llvm-project/pull/109068
More information about the cfe-commits
mailing list