----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102050/#review4972 -----------------------------------------------------------
I don't think your premise for doing this isn't entirely true. When you load a QPixmap from a file, it is automatically added to an implicit QPixmapCache Relevant: http://doc.trolltech.com/4.7/qpixmap.html#load - David On July 22, 2011, 2:42 p.m., Dario Freddi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102050/ > ----------------------------------------------------------- > > (Updated July 22, 2011, 2:42 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > At the moment, each time an avatar is requested, a new pixmap from the image > file is created. This is of course more than evil, and this patch aims to fix > that. PixmapCache is a proxy to QPixmapCache for handling avatars' pixmaps > efficiently. If this patch gets in, from now on every avatar consumer > (tooltips, list view, etc) should get his avatar pixmaps through the cache. > Of course, in the future, this will be extended to support the most common > and most unefficiently handled pixmaps in our code. > > > Diffs > ----- > > CMakeLists.txt de2827f > contact-delegate-compact.cpp 45cd282 > contact-delegate.cpp 159894f > pixmap-cache.h PRE-CREATION > pixmap-cache.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/102050/diff > > > Testing > ------- > > Apparently everything still works > > > Thanks, > > Dario > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
