Prazek added a subscriber: Prazek.
================
Comment at: clang-tidy/misc/EnumMisuseCheck.cpp:19
@@ +18,3 @@
+
+// Stores a min and a max value which describe an interval.
+struct ValueRange {
----------------
s/\/\//\/\/\/
In other words, change // to /// (doxygen comment)
================
Comment at: clang-tidy/misc/EnumMisuseCheck.cpp:192
@@ +191,3 @@
+ const auto *LhsConstant =
+ LhsDecExpr ? dyn_cast<EnumConstantDecl>(LhsDecExpr->getDecl()) : nullptr;
+ const auto *RhsConstant =
----------------
If the pointer returned from dyn_cast<EnumConstantDecl>(LhsDeclExpr->getDecl())
is assumed to be always not null, thn you can change it to cast<> (It will
assert if null instead of returning nullptr)
================
Comment at: docs/clang-tidy/checks/misc-enum-misuse.rst:29
@@ +28,3 @@
+
+.. code:: c++
+
----------------
I think all the code here needs to be indented by one or more to get into
..code:: block.
https://reviews.llvm.org/D22507
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
