danielmarjamaki added inline comments. ================ Comment at: clang-tidy/misc/MisplacedWideningCastCheck.cpp:79 @@ +78,3 @@ + +void MisplacedWideningCastCheck::check(const MatchFinder::MatchResult &Result) { + const auto *Cast = Result.Nodes.getNodeAs<ExplicitCastExpr>("Cast"); ---------------- I looked in another checker , that did not have a condition nor assert so I removed it.
================ Comment at: clang-tidy/misc/MisplacedWideningCastCheck.h:17 @@ +16,3 @@ +namespace tidy { +namespace misc { + ---------------- I have rerun add_new_check.py for my latest patch. ================ Comment at: docs/clang-tidy/checks/misc-misplaced-widening-cast.rst:5 @@ +4,3 @@ +============================ + +This check will warn when there is a explicit redundant cast of a calculation ---------------- yes good catch. http://reviews.llvm.org/D16310 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits