Review Request 109834: Expose sender in KTp::Message

Commit Hook null at kde.org
Thu Apr 18 01:30:06 UTC 2013


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

(Updated April 18, 2013, 1:30 a.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Description
-------

Expose senderAvatar in KTp::Message

Simplify code and make it less possible to crash.
Store the contactPtr of the sender where possible to avoid storing the samepixmap many times.

Also fix message direction of log messages.


Diffs
-----

  KTp/message.h 812c90e6ddf6d77af9ffab1f866be57fe95419aa 
  KTp/message.cpp cbb91346bc174ebbab279b2eae732a1e322b5a7e 

Diff: http://git.reviewboard.kde.org/r/109834/diff/


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130418/3c35ad93/attachment-0001.html>


More information about the KDE-Telepathy mailing list