[PATCH] D19201: [clang-tidy] misc-throw-with-noexcept

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 16 13:20:52 PST 2017


Prazek added inline comments.


================
Comment at: clang-tidy/misc/ThrowWithNoexceptCheck.cpp:52-53
+  for (const auto &NoExceptRange : NoExceptRanges) {
+    // FIXME use DiagnosticIDs::Level::Note
+    diag(NoExceptRange.getBegin(), "In function declared no-throw here:")
+        << FixItHint::CreateRemoval(NoExceptRange);
----------------
just add DiagnosticIDs::Note as third argument
This might also change the order of diagnostics as we discussed ofline


https://reviews.llvm.org/D19201





More information about the cfe-commits mailing list