[Kde-pim] Review Request: Cache KDateTime in SocialFeedItem

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107113/#review21478
-----------------------------------------------------------


This review has been submitted with commit 50d1767155efaeecbc1d1d9194dbdc8a3da04241 by Martin Klapetek to branch master.

- Commit Hook


On Oct. 29, 2012, 2:28 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107113/
> -----------------------------------------------------------
> 
> (Updated Oct. 29, 2012, 2:28 p.m.)
> 
> 
> Review request for KDEPIM and KDEPIM-Libraries.
> 
> 
> Description
> -------
> 
> SocialFeedItem::postTime() is accessed directly from the model and it's called a lot, putting the plasmoid through callgrind shows that creating the KDateTime everyone it's called is quite expensive, so now it's created just once and the existing KDateTime is returned.
> 
> I also removed the setters for postTimeString and postTimeFormat and left only one setter for setting them both at once, creating the KDateTime in that method as it needs both the format and string.
> 
> 
> Diffs
> -----
> 
>   akonadi/socialutils/serializer/akonadi_serializer_socialfeeditem.cpp a153d92 
>   akonadi/socialutils/socialfeeditem.h 2cbabe9 
>   akonadi/socialutils/socialfeeditem.cpp 6d6572b 
> 
> Diff: http://git.reviewboard.kde.org/r/107113/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/