[PATCH] D23728: [Clang-tidy] Fix style in some checks documentation

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 20 10:01:08 PDT 2016


alexfh requested changes to this revision.
This revision now requires changes to proceed.

================
Comment at: docs/clang-tidy/checks/cppcoreguidelines-pro-bounds-constant-array-index.rst:11
@@ -10,3 +10,3 @@
 
-The check can generate fixes after the option `GslHeader` has been set
+The check can generate fixes after the :option:`GslHeader` has been set
 to the name of the include file that contains ``gsl::at()``, e.g. `"gsl/gsl.h"`.
----------------
I don't see the rest of the file (upload full diffs, btw), but we need to add `.. option:: GslHeader`, if it's not there yet. And I think, the general rule should be to add option description in every check that uses the option, since different checks can use the same global option in a subtly different ways.

I hope, Sphinx supports this use case.


Repository:
  rL LLVM

https://reviews.llvm.org/D23728





More information about the cfe-commits mailing list