----------------------------------------------------------- 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 ------- g...@coherenttheory.com:plasmoidprogress.git Thanks, Aaron
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
