[PATCH] [analyzer][Review request] Improved checker lists.

Антон Ярцев anton.yartsev at gmail.com
Tue Apr 22 15:02:17 PDT 2014


Hi jordan_rose,

Here are the renewed checker lists we started to switch to a long time ago. The last suggestion was to split examples to produce one warning per example. Attached patch addresses this. It also contain lots of new improvements. Since the last review:
- made the list of actual checkers to look like the list of potential checkers, added examples;
- tested, reviewed and updated examples, removed odd and outdated ones, added new;
- split examples to produce a single warning per example;
- improved html structure and markup;
- added the improved expand/collapse feature;
- reviewed and improved descriptions, added missing <code> tegs, added missing periods;
- pointed to the sources related to the problem the checker evolves;

http://reviews.llvm.org/D3457

Files:
  www/analyzer/available_checks.html
  www/analyzer/checker_dev_manual.html
  www/analyzer/content.css
  www/analyzer/potential_checkers.html

REPLY HANDLER ACTIONS
  Reply to comment, or !reject, !abandon, !reclaim, !resign, !rethink, !unsubscribe.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3457.8743.patch
Type: text/x-patch
Size: 134249 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140422/33a393c5/attachment.bin>


More information about the cfe-commits mailing list