alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
LG with a couple of nits. Do you need me to submit the patch for you? (If yes, I can fix the doc myself) ================ Comment at: docs/clang-tidy/checks/llvm-header-guard.rst:8 @@ +7,3 @@ + +For the user-facing documentation see: http://clang.llvm.org/extra/clang-tidy/checks/llvm-header-guard.html + ---------------- This line is not needed here (it's a self-reference). ================ Comment at: docs/clang-tidy/checks/llvm-header-guard.rst:10 @@ +9,3 @@ + +The check supports these options: +``HeaderFileExtensions``: a comma-separated list of filename extensions of header files (The filename extension should not contain "." prefix). ",h,hh,hpp,hxx" by default. For extension-less header files, using an empty string or leaving an empty string between "," if there are other filename extensions. ---------------- Should be Options ------- .. option:: HeaderFileExtensions ... https://reviews.llvm.org/D20512 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits