berenm added inline comments. ================ Comment at: clang-tidy/readability/IdentifierNamingCheck.cpp:508 @@ +507,3 @@ + auto &Failure = Failures[Decl]; + for (const auto &R : Failure.Usages) { + if (R == Range) ---------------- Hopefully the number of ranges in Usages will stay low. If not, `Usages` should be an hash table instead, but it looks like SourceRange isn't hashable as is.
http://reviews.llvm.org/D13079 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits