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

Malcolm Parsons via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 6 08:59:18 PST 2017


malcolm.parsons added inline comments.


================
Comment at: clang-tidy/modernize/UseNoexceptCheck.h:19
+
+using CandidateSet = llvm::StringSet<llvm::MallocAllocator>;
+
----------------
Unused?


================
Comment at: docs/clang-tidy/checks/modernize-use-noexcept.rst:38
+
+Please note that since ``throw(int)`` is equivelent to
+``noexcept(false)`` not ``noexcept``, this check will detect, but not
----------------
typo: equivelent -> equivalent.


https://reviews.llvm.org/D20693





More information about the cfe-commits mailing list