Review Request 123209: export connectionstatus remove isChangingPresence

[David Edmundson]

> On April 1, 2015, 2:25 p.m., Martin Klapetek wrote:
> > This breaks ABI...?
> > 
> > Also, is it not possible to just register the Tp type directly with QML?

Yes it does break ABI, but is that a problem?

I don't think using the TpQt enum is possible, not without changing TpQt.

The best we could possibly do is export it in the property as Tp::ConnectionStatus, then rely on the fact that QML just treats it all as an int and using some other enum on the QML side.

or I could just keep it as a boolean accessor like I did in the 15.04 branch.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123209/#review78339
-----------------------------------------------------------


On April 1, 2015, 1:29 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123209/
> -----------------------------------------------------------
> 
> (Updated April 1, 2015, 1:29 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-common-internals
> 
> 
> Description
> -------
> 
> Not sure I like it, I have to shadow an enum which gets messy. Dunno.
> 
> 
> Diffs
> -----
> 
>   KTp/global-presence.h 61a35bb 
>   KTp/global-presence.cpp d47a695 
> 
> Diff: https://git.reviewboard.kde.org/r/123209/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20150401/25fc9c98/attachment.html>