Review Request: fewer pixmaps created in pixmap transitions

Aaron Seigo aseigo at kde.org
Wed Sep 22 20:36:21 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5406/
-----------------------------------------------------------

(Updated 2010-09-22 18:36:20.575729)


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 (updated)
-----

  /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/49ab54c5/attachment-0001.htm 


More information about the Plasma-devel mailing list