aaron.ballman added inline comments.

================
Comment at: lib/StaticAnalyzer/Checkers/EnumCastOutOfRangeChecker.cpp:19
+//   of casting an integer value that is out of range
+//===----------------------------------------------------------------------===//
+
----------------
If this check is intended to conform to CERT's INT50-CPP rule, you should put a 
link to the wiki entry for it here as well.


================
Comment at: lib/StaticAnalyzer/Checkers/EnumCastOutOfRangeChecker.cpp:72
+      std::distance(ED->enumerator_begin(), ED->enumerator_end()));
+  std::transform(ED->enumerator_begin(), ED->enumerator_end(),
+                 DeclValues.begin(),
----------------
You can use `llvm::transform(ED->enumerators(), ...)` instead -- the semantics 
are identical to `std::transform()`, but it takes a range instead of a pair of 
iterators.


https://reviews.llvm.org/D33672



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to