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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to