alexfh added a comment.
A few late comments.
================
Comment at:
clang-tools-extra/trunk/docs/clang-tidy/checks/readability-static-accessed-through-instance.rst:28-29
+
+ C::foo();
+ C::x;
+
----------------
This may be confusing as to whether the check removes the struct definition
(and the definition of c1) or not.
================
Comment at:
docs/clang-tidy/checks/readability-static-accessed-through-instance.rst:11
+
+Thefollowing code:
+
----------------
typo: "Thefollowing"
================
Comment at: test/clang-tidy/readability-static-accessed-through-instance.cpp:55
+ // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: static member accessed through
+ // instance [readability-static-accessed-through-instance]
+ // CHECK-FIXES: {{^}} C::x;{{$}}
----------------
Line wrapping gone bad. I guess you can just remove the second line. Same below.
Repository:
rL LLVM
https://reviews.llvm.org/D35937
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
