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

[don hinton via Phabricator via cfe-commits]

hintonda added inline comments.


================
Comment at: clang-tidy/modernize/UseNoexceptCheck.h:19
+
+using CandidateSet = llvm::StringSet<llvm::MallocAllocator>;
+
----------------
malcolm.parsons wrote:
> Unused?
Good catch -- left over from a previous version.


================
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
----------------
malcolm.parsons wrote:
> typo: equivelent -> equivalent.
Thanks!  Spelling isn't my strong suit.


https://reviews.llvm.org/D20693