[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