[PATCH] D53069: [analyzer][www] Update avaible_checks.html
George Karpenkov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 10 10:46:51 PDT 2018
george.karpenkov requested changes to this revision.
george.karpenkov added a comment.
This revision now requires changes to proceed.
Herald added a subscriber: donat.nagy.
Great idea, thanks!
Should be good to go once examples are added, and implicit checks are removed.
================
Comment at: www/analyzer/available_checks.html:770
+(ObjC)</span><div class="descr">
+Check for proper uses of Objective-C properties</div></div></td>
+<td><div class="exampleContainer expandable">
----------------
`proper uses` is not particularly descriptive?
================
Comment at: www/analyzer/available_checks.html:884
+<div class="example"><pre>
+<!-- TODO: Add an example. -->
+</pre></div></div></td></tr>
----------------
Yep, not very useful without an example. I'm pretty sure the code and tests have one.
================
Comment at: www/analyzer/available_checks.html:988
+<!-- TODO: Is this an implicit checker? -->
+<tr><td><div class="namedescr expandable"><span class="name">
----------------
Yeah, doesn't seem useful to the user.
https://reviews.llvm.org/D53069
More information about the cfe-commits
mailing list