alexfh added a comment. A high-level comment: the "misc" module is the place for checks that we didn't find (or create) a better category for. Here it's clear that we need a separate category, so we need a `CppCoreGuidelinesModule` and the `cppcoreguidelines-` check prefix. I also suggest using anchor names (converted to lower case) in the document as check names, this one would be `cppcoreguidelines-pro-type-reinterpretcast`.
================ Comment at: test/clang-tidy/misc-no-reinterpret-cast.cpp:7 @@ +6,1 @@ +// CHECK-MESSAGES: :[[@LINE-1]]:16: warning: do not use reinterpret_cast (C++ Core Guidelines, rule Type.1) [misc-no-reinterpret-cast] \ No newline at end of file ---------------- Please add a newline at the end of file to pacify various diff tools. http://reviews.llvm.org/D13313 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
