[PATCH] D29839: [clang-tidy] New misc-istream-overflow check

[Kostya Kortchinsky via Phabricator via cfe-commits]

cryptoad marked 5 inline comments as done.
cryptoad added inline comments.


================
Comment at: clang-tidy/misc/IstreamOverflowCheck.cpp:78-80
+    if (!Arg->isIntegerConstantExpr(WidthSize, Context)) {
+      llvm::errs() << "Couldn't get width() size.\n";
+    }
----------------
Prazek wrote:
> debug?
Oops!
I'd like to somehow keep that in there (for debug purposes). Any preferred way> 
(ifdef DEBUG or other).


Repository:
  rL LLVM

https://reviews.llvm.org/D29839



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