Plasma::Svg and performance

[Marco Martin]

On Wednesday 15 October 2008, Aaron J. Seigo wrote:
>
> > (maybe imited to a fixed number of
> > elements say 100 to not make it grow forever)
>
> it will need to be a fixed size, yes, even without non-existent elements.
> 100 is probably too small though. this isn't a huge amount of data, really.
>
> the actual problem with the patch is that it doesn't actually buy us
> anything: once the renderer is created, we've lost. so it's the right
> start, just needs the final bit to actually make your work here meaningful
> =)
with a kconfigroup? would make sense?

> the cache will need to be read from disk at startup and written to disk at
> shutdown so that we can take advantage of previous look ups.
>
> that way after the first run (or a new theme gets set) we can prevent
> creation of QSvgRenderer objects (which easily lead to 1-2MB of allocation
> for parsing)
>
> i'd suggest putting the cache in the Theme object, just as the pixmap cache
> is.