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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits