On Fri, Nov 13, 2015 at 10:45 PM, Alexander Kornienko <ale...@google.com> wrote:
> alexfh accepted this revision.
> alexfh added a comment.
> This revision is now accepted and ready to land.
>
> Looks good with one nit. Thank you!
>
>
> ================
> Comment at: clang-tidy/cert/ThrownExceptionTypeCheck.cpp:23
> @@ +22,3 @@
> + if (const auto *FnTy = Node.getType()->getAs<FunctionProtoType>()) {
> + // If any of the copy constructors is throwing, we will assume that it
> + // is plausible that the constructor may be selected by the throw and
> ----------------
> The comment is not relevant now.
Thanks! I've commit in r253246.
~Aaron
>
>
> http://reviews.llvm.org/D14619
>
>
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
