[PATCH] D20693: [clang-tidy] New checker to replace dynamic exception specifications

Matthias Gehre via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 6 11:23:00 PST 2017


mgehre added a comment.

Hi,
this is a good idea for a check.

I would prefer when the FixIt just removes throw(A,B) specifier, instead of replacing it by noexcept(false),
because noexcept(false) means the same things as having no noexcept specifier at all.
And less code to read means its easier to understand.


https://reviews.llvm.org/D20693





More information about the cfe-commits mailing list