[PATCH] D40787: [clang-tidy] Replace the usage of std::uncaught_exception with std::uncaught_exceptions

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 3 06:59:56 PST 2018


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

This LGTM with a few minor nits to fix.



================
Comment at: clang-tidy/modernize/ModernizeTidyModule.cpp:80-81
     CheckFactories.registerCheck<UseOverrideCheck>("modernize-use-override");
+    CheckFactories.registerCheck<UseUncaughtExceptionsCheck>(
+        "modernize-use-uncaught-exceptions");
     CheckFactories.registerCheck<UseTransparentFunctorsCheck>(
----------------
Please keep this list alphabetized.


================
Comment at: docs/clang-tidy/checks/modernize-use-uncaught-exceptions.rst:19
+    int uncaught_exception() {
+        return 0;
+    }
----------------
The indentation of the code examples (here and below) is incorrect.


https://reviews.llvm.org/D40787





More information about the cfe-commits mailing list