[PATCH] D15332: new clang-tidy checker readability-non-const-parameter

Daniel Marjamäki via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 10 04:49:25 PDT 2016


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





More information about the cfe-commits mailing list