[PATCH] D21020: [clang-tidy] readability-identifier-naming - Support for Macros

James Reynolds via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 8 10:03:34 PDT 2016


JamesReynolds added inline comments.

================
Comment at: clang-tidy/readability/IdentifierNamingCheck.h:89
@@ +88,3 @@
+
+  typedef std::pair<SourceLocation, std::string> NamingCheckId;
+
----------------
I think I thought that this wouldn't get picked up by the specialization machinery - but you're right, a typedef works out fine! Maybe I was thinking of overloads...


http://reviews.llvm.org/D21020





More information about the cfe-commits mailing list