[PATCH] D54429: [analyzer] Creating standard Sphinx documentation
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 31 12:32:56 PST 2019
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
I think let's get this up and running and slowly redirect links from the front page to the new docs until only the front page is left?
================
Comment at: docs/analyzer/checkers.rst:36-37
+
+.. literalinclude:: checkers/callandmessage_example.c
+ :language: objc
+
----------------
Wow, that's pretty cool. Can we actually merge documentation with regression tests somehow, so that to automatically ensure that the documentation is correct?
================
Comment at: docs/analyzer/checkers.rst:2003
+View Exploded Graphs using GraphViz.
+
----------------
Szelethus wrote:
> While I would argue very strongly against the current website's every effort at hiding implicit checkers, when we deliberately call this site a documentation site, I definitely think that we should most include them here.
>
> Although, don't sweat it too much just yet, while the structure is still being decided upon.
Well, i mean, it's a good idea to make a distinction between a user guide from a developer guide. We should totally document these checkers, but we want to keep the user part of the guide user-comprehendable and, ideally, as short as possible. It's like people don't usually combine code comments with man pages; both are "documentation", but there's a huge difference between them driven by different target audience.
At the same time, we already have a better "Debug Checks" page.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54429/new/
https://reviews.llvm.org/D54429
More information about the cfe-commits
mailing list