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