[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