On Wednesday 15 October 2008, Jamboarder wrote:
> > From: Aaron J. Seigo <[EMAIL PROTECTED]>
> >
> >> setImagePath (if the path is different that the current one)
> >
> > this only affects Plasma::SharedSvgRenderer usage, and is local to any
> > given Plasma::Svg object.
>
> Might the pixmap cache end up holding data for svg paths that are no longer
> being used by any instance of Plasma::Svg?

which is ok; there's a max size enforced and older items simply get pushed out 
of the cache.

> (perhaps this was the memory
> leak you mentioned).

no, the leak is that it wasn't deleting the KPixmapCache object .. ever. ;)

> > yes, in Plasma::Theme. why? it knows when the theme parameters change.
>
> Ah, that make sense. :-)  Although it does raise one additional question
> for me.  I'm not sure if it is the intended use but is it
> possible/desirable that Plasma::Svg might be used without Plasma::Theme
> (for example to render non-theme-related svg files).

already can... but in that case, theme parameters don't matter =)

> In any case are theme parameter changes the only vector for changes that
> might warrant a clear/refresh of svg related cache data? 

yes.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to