NoQ marked an inline comment as done.
NoQ added inline comments.

================
Comment at: clang/utils/analyzer/exploded-graph-rewriter.py:326
+            self._dump('<b>Program point:</b></td></tr>')
+        self._dump('<tr><td align="left" balign="left" width="0">'
+                   '<table border="0" align="left" width="0">')
----------------
Charusso wrote:
> I would create a table-builder class to reduce the repetition and for better 
> readability what is going on. Also you could avoid `balign` typo.
> 
> Then may each class could have its own pretty-print method like LLVM does.
`balign` is a thing in the graphviz dialect of html 
(https://www.graphviz.org/doc/info/shapes.html)

(not sure i still need this in the current incarnation of the script, given 
that we've removed those whole-decl pretty-prints in D62495)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62638/new/

https://reviews.llvm.org/D62638



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

Reply via email to