scan-view migrated from optparse deprecated Python module to its replacement (argparse) and resolved few conflicts with pep8
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Looks reasonable to me from a "I haven't actually tried it with this patch" perspective.
Comment Actions
Overall, looks good!
tools/scan-view/bin/scan-view | ||
---|---|---|
94 ↗ | (On Diff #42272) | Please, add the tool description, like "The clang static analyzer results viewer." |