[Telepathy] API Draft for high level tubes in tp-qt4
drf54321 at gmail.com
Thu Jun 3 00:10:00 PDT 2010
On Thursday 03 June 2010 06:51:20 you wrote:
> We talked this through over IRC, with the end result being that for
> connection tracking, we should have something like:
> UIntList connections(); // list of current connection ids
> signal connectionAdded(uint connectionId);
> signal connectionClosed(uint connectionId, QString error, QString message);
Oh my. Definitely the two signals should be merged. Will do after your review.
> ContactPtr contactForConnection(uint validConnectionId); // always
> available as long as connection tracking is enabled
> uint connectionForSourceAddress(QPair<QHostAddress, uint>
> addressAndPort); // available if
> supportsIP*SocketsWithAllowedAddress() is true, with the lib
> auto-specifying Port access control on offer() if available
> I think having these mapping functions is better than returning a QHash,
Hmm, after all they're an implementation of an hash with lookups by key. I
think returning a QHash is more sensible here: it does not clutter the API and
still provides developers a more comfortable way for handling those. Anyway,
have a look at how I implemented it.
> nothing :)
> Come to think of it, we should probably rename
> supportsIP*SocketsWithAllowedAddress to *WithSpecifiedAddress - fits
> more naturally into connectionForSourceAddress
Well, I am very bad at namings, so when you'll have a look at the code just
confirm you want to change this, and I will :)
> Will take a look at your latest commits (probably contradicting this
> recap because of some oversight of mine :)) today evening.
GPG Key Signature: 511A9A3B
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: This is a digitally signed message part.
More information about the telepathy