[PATCH] D65250: [analyzer] exploded-graph-rewriter: Improve user-friendliness.

Thu Jul 25 08:12:32 PDT 2019

Charusso added a comment.

In D65250#1601187 <https://reviews.llvm.org/D65250#1601187>, @NoQ wrote:

> In D65250#1600776 <https://reviews.llvm.org/D65250#1600776>, @grandinj wrote:
>
> > There is no need to wrap SVG in HTML if you want to display it in a web-browser
>
>
> I guess it's worth it when the recipient doesn't trust html attachments (eg., for sending to the mailing lists).

You could put HTML inside the SVG (https://developer.mozilla.org/en-US/docs/Web/SVG/Element/foreignObject). That is a cool feature just I have not managed to use it yet, but I would use it for better performance in loading the SVG, and that definitely needs an HTML wrapper.

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

https://reviews.llvm.org/D65250