[PATCH] D65250: [analyzer] exploded-graph-rewriter: Improve user-friendliness.
Csaba Dabis via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 26 13:03:39 PDT 2019
Charusso accepted this revision.
Charusso added a comment.
This revision is now accepted and ready to land.
I like the HTML output, thanks!
================
Comment at: clang/utils/analyzer/exploded-graph-rewriter.py:846
+ print(' $ dot -Tsvg input.dot -o output.svg')
+ print()
+ write_temp_file('.dot', self.output())
----------------
Why we need multiple prints to print one message? According to the Stack Overflow:
```
print("""
Line1
Line2
""")
```
Link: https://stackoverflow.com/questions/34980251/how-to-print-multiple-lines-of-text-with-python
================
Comment at: clang/utils/analyzer/exploded-graph-rewriter.py:953
+ 'displaying it, dump the rewritten dot file '
+ 'to stdout')
args = parser.parse_args()
----------------
This flag is a little-bit too long and does not really emphasize what it suppose to do. What about just `--dump` or `--stdout`?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65250/new/
https://reviews.llvm.org/D65250
More information about the cfe-commits
mailing list