Review Request: fewer pixmaps created in pixmap transitions
Aaron Seigo
aseigo at kde.org
Wed Sep 22 18:44:25 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5406/
-----------------------------------------------------------
Review request for Plasma.
Summary
-------
This prevents the creation of new pixmaps as much as possible while doing transition effects at the cost of creating more pixmaps on the fly. It introduces a new property, cache, to PixmapTransition to restore the old behaviour. However, most of our widgets already do caching internally, so the added pixmap copy is just that: a copy. It sits around taking up pixmap memory for no particularly good reason. In the Javascript plasmoid I'm working on, it creates a table with buttons, some 146 buttons, in fact. This drops the Pixmap count from nearly 150 for the same (pixel-wise) button pixmap to 1.
Combined with my commits from last night and a new caching scheme for FrameSvg, creating these buttons has dropped from 6-9 seconds on my machine to 2-3 seconds.
Diffs
-----
/trunk/KDE/kdelibs/plasma/animations/pixmaptransition.cpp 1178174
/trunk/KDE/kdelibs/plasma/animations/pixmaptransition_p.h 1178174
/trunk/KDE/kdelibs/plasma/paintutils.cpp 1178174
Diff: http://svn.reviewboard.kde.org/r/5406/diff
Testing
-------
git at coherenttheory.com:plasmoidprogress.git
Thanks,
Aaron
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100922/d02006b0/attachment.htm
More information about the Plasma-devel
mailing list