[Bug 32612] Remove Tubes code
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Wed Jul 18 22:01:52 CEST 2012
https://bugs.freedesktop.org/show_bug.cgi?id=32612
--- Comment #14 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-07-18 13:01:52 PDT ---
"muc-channel: disappear from the bus when not requested"
+ DEBUG ("making MUC channel reappear!");
+ tp_base_channel_reopened_with_requested (base, FALSE, sender);
+ g_signal_emit (chan, signals[APPEARED], 0);
I can't help thinking that if the appeared signal is necessary in Gabble, then
telepathy-glib is not being sufficiently helpful for Bug #48210.
Couldn't muc_channel_closed_cb handle the reappearance if
(channel.is_respawning()) like I suggested in Bug #48210 Comment #6? (Yes, I
realise this is pretty strange.)
Alternatively, TpExportableChannel or TpBaseChannel could grow a reappeared
signal or something?
+ gboolean needed_not_wanted,
If you reverse the sense of this flag, and call it export_text or something,
then things might be a bit more natural:
+ if (gabble_muc_channel_get_autoclose (*ret))
+ gabble_muc_channel_set_autoclose (*ret, needed_not_wanted);
could become
if (export_text)
gabble_muc_channel_set_autoclose (FALSE);
which I think is maybe clearer.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
More information about the telepathy-bugs
mailing list