[PATCH] D30406: [Analyzer] Add support for displaying cross-file diagnostic paths in HTML output
Anna Zaks via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 1 13:07:58 PST 2017
zaks.anna added a comment.
No multi-file support is a long outstanding limitation of scan-build html output. Great to see the patch!! Thank you for working on it!
> It's not as immediately clear this is a multi-file output.
In addition to Artem's suggestions, you might want to insert multiple lines of padding to make the distinction on the border more clear. I think it would help especially when scrolling a large report like in the link for the Linux source.
Also, could you put this behind an option or introduce a new format like -analyzer-output=plist-multi-file but for html? Just in case someone is relying on a single file output format, we'd want to have an option to give them to turn it on.
https://reviews.llvm.org/D30406
More information about the cfe-commits
mailing list