[PATCH] D16310: new clang-tidy checker misc-long-cast

Richard via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 9 22:28:52 PST 2016


LegalizeAdulthood added inline comments.

================
Comment at: clang-tools-extra/trunk/clang-tidy/misc/MisplacedWideningCastCheck.cpp:21-23
@@ +20,5 @@
+void MisplacedWideningCastCheck::registerMatchers(MatchFinder *Finder) {
+  auto Calc = expr(anyOf(binaryOperator(anyOf(
+                             hasOperatorName("+"), hasOperatorName("-"),
+                             hasOperatorName("*"), hasOperatorName("<<"))),
+                         unaryOperator(hasOperatorName("~"))),
----------------
Sorry for the late observation, but why doesn't this check for `%` and `/` operators?


Repository:
  rL LLVM

http://reviews.llvm.org/D16310





More information about the cfe-commits mailing list