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