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

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 24 09:40:59 PDT 2018


Szelethus marked 14 inline comments as done.
Szelethus added inline comments.


================
Comment at: www/analyzer/available_checks.html:459
 
+
 </tbody></table>
----------------
george.karpenkov wrote:
> Spurious newline
Actually, in this section of the code, entries are separated with 2 newlines. But it's not super consistent.


================
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>
----------------
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.


https://reviews.llvm.org/D53069





More information about the cfe-commits mailing list