vbvictor wrote:

> CastSequence (UseNullptrCheck.cpp, line 24) could be made static and moved 
> out of the anonymous namespace

Yeah we can, but technically it isn't flagged rn because 
[option](https://clang.llvm.org/extra/clang-tidy/checks/llvm/prefer-static-over-anonymous-namespace.html#cmdoption-arg-AllowVariableDeclarations)
 is set to `true` by default

https://github.com/llvm/llvm-project/pull/164085
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to