[Bug 46513] Make Jingle code less Telepathic
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Wed Feb 29 14:23:41 CET 2012
https://bugs.freedesktop.org/show_bug.cgi?id=46513
--- Comment #7 from Will Thompson <will.thompson at collabora.co.uk> 2012-02-29 05:23:41 PST ---
(In reply to comment #5)
> > + /* Only emitted for new incoming sessions, mainly for legacy reasons */
> > + signals[NEW_SESSION] = g_signal_new ("new-session",
>
> Can't the JingleMint's new signal be "incoming-session" or something?
Good idea. Renamed.
> In da9f352766e "Represent Jingle relays by a struct, not an aa{sv}":
>
> > + GPtrArray *tp_relays = gabble_build_tp_relay_info (relays);
> > + tp_base_media_call_stream_set_relay_info (stream, tp_relays);
>
> Nitpicking: I'd prefer a blank line between initialized locals and other code.
Fixed.
--
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