Review Request: Add static methods to retrieve icon, iconName and displaystring from a Tp::ConnectionPresenceType to KTp presence

[David Edmundson]

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

Ship it!


Code is absolutely fine, especially the documentation.

I would like to know the reason we need this though. I'm not aware of anywhere in TpQt that gives you a Tp::ConnectionPresenceType.

I don't want people to start writing:
Ktp::Presence::iconName(myPresence.connectionPresenceType())

instead of the current:
KTp::Presence(myPresence).iconName();

Drop me a message sometime (XMPP or IRC).

- David Edmundson


On Feb. 24, 2012, 6:01 p.m., Daniele Elmo Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104069/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2012, 6:01 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Add static methods to retrieve icon, iconName and displaystring from a Tp::ConnectionPresenceType to KTp presence and use them internally
> 
> 
> Diffs
> -----
> 
>   KTp/presence.h d116530c318586ef290f7193c9f1956f4e47b77d 
>   KTp/presence.cpp 35bc0dc6088b875bf719a22f45938184b1f96bb2 
> 
> Diff: http://git.reviewboard.kde.org/r/104069/diff/
> 
> 
> Testing
> -------
> 
> Built, installed, and run components using them.
> Everything seems ok.
> 
> 
> Thanks,
> 
> Daniele Elmo Domenichelli
> 
>

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