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

don hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 10 16:46:50 PST 2017


hintonda added inline comments.


================
Comment at: docs/clang-tidy/checks/modernize-use-noexcept.rst:6-8
+The check converts dynamic exception specifications, e.g.,
+``throw()``, ``throw(<exception>[,...])``, or ``throw(...)``, to
+``noexcept``, ``noexcept(false)``, blank, or a user defined macro.
----------------
alexfh wrote:
> alexfh wrote:
> > This description doesn't say why `noexcept` is better.
> This still needs to be addressed.
Absolutely, but I'll probably do all the code changes first while it's still fresh in my mind. 


https://reviews.llvm.org/D20693





More information about the cfe-commits mailing list