[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