danielmarjamaki marked 2 inline comments as done. ================ Comment at: test/clang-tidy/readability-non-const-parameter.cpp:245 @@ +244,3 @@ + C c(p); +} + ---------------- I have added tests and fixed FPs in latest diff.
================ Comment at: test/clang-tidy/readability-non-const-parameter.cpp:252 @@ +251,3 @@ + +class Warn { +public: ---------------- I have added a test in latest diff. but there is a false positive. I believe I heard about whole program analysis in clang-tidy sometime.. is that something worked on / implemented yet? https://reviews.llvm.org/D15332 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits