Review Request: Implement a cache pixmap for avatars

[David Edmundson]

-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110722/d5aa4267/attachment-0001.htm