[clang-tools-extra] [clang-tidy] Filter out googletest TUs in bugprone-unchecked-optional-access (PR #115051)
Carlos Galvez via cfe-commits
cfe-commits at lists.llvm.org
Sat Dec 7 03:52:48 PST 2024
carlosgalvezp wrote:
> It is normal cases that source code and test code have different quality metrics.
+1. That's responsibility of the build system. "What is a test" is something that can vary from project to project.
> is expected to be removed again
>From experience, removing flags it's quite hard, due to backwards compatibility reasons (we break other people that rely on them). It usually takes about 2 releases (1 year) to pull it off.
So I would be strongly against adding such a flag now if we already see that it will be removed - we should rather take the time to implement this right from the start.
https://github.com/llvm/llvm-project/pull/115051
More information about the cfe-commits
mailing list