[PATCH] D52984: [analyzer] Checker reviewer's checklist

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 10 07:03:28 PST 2018


Szelethus added inline comments.


================
Comment at: www/analyzer/checker_dev_manual.html:719
+<ul>
+<li>User facing documentation is important for adoption! Make sure the check list updated
+    at the homepage of the analyzer. Also ensure that the description is good quality in
----------------
xazax.hun wrote:
> Szelethus wrote:
> > Make sure the **checker** list **is** updated
> I think at some point we should decide if we prefer the term check or checker to refer to these things :)  Clang Tidy clearly prefers check.
That is the distinction I'm aware of too: checkers in the Static Analyzer, checks in clang-tidy.


https://reviews.llvm.org/D52984





More information about the cfe-commits mailing list