[Bug 46484] Add high-level Call bindings
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Thu Mar 15 10:01:34 CET 2012
https://bugs.freedesktop.org/show_bug.cgi?id=46484
--- Comment #7 from Dario Freddi <drf54321 at gmail.com> 2012-03-15 02:01:34 PDT ---
(In reply to comment #4)
> (In reply to comment #2)
>
> >
> > + Q_ASSERT(mPriv->remoteMembersContacts.contains(handle));
> >
> > I guess we don't want to assert here
>
> Why not? It's a condition that should be met at this point. Saves us from
> accidentally creating a null ContactPtr without noticing (when calling
> mPriv->remoteMembersContacts[handle] below).
Sure, my point is that instead of asserting we should return an error - we
don't want the application to crash if shit happens.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the telepathy-bugs
mailing list