[PATCH] D53069: [analyzer][www] Update avaible_checks.html

George Karpenkov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 24 10:24:46 PDT 2018


george.karpenkov requested changes to this revision.
george.karpenkov added a comment.
This revision now requires changes to proceed.

Good to go provided you will add an example.
If we want to be serious about this page, it really has to be auto-generated (like clang-tidy one), but I understand that this is a larger undertaking.



================
Comment at: www/analyzer/available_checks.html:496
+<div class="example"><pre>
+<!-- TODO: Add examples. And also test files for this checker. -->
+</pre></div></div></td></tr>
----------------
Szelethus wrote:
> george.karpenkov wrote:
> > If we don't have a description, let's just drop it.
> I added a description, I'd strongly prefer keeping this in.
Also would need a short example. Usually one can be found in tests.


https://reviews.llvm.org/D53069





More information about the cfe-commits mailing list