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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to