[Telepathy-commits] [telepathy-gabble/master] GabbleMucFactory: add a comment to explain why we use a temporary variable

Alban Crequy alban.crequy at collabora.co.uk
Thu Nov 27 11:00:37 PST 2008


---
 src/muc-factory.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/muc-factory.c b/src/muc-factory.c
index 8f17f8e..f294a91 100644
--- a/src/muc-factory.c
+++ b/src/muc-factory.c
@@ -1056,6 +1056,9 @@ gabble_muc_factory_close_all (GabbleMucFactory *self)
       priv->text_needed_for_tubes = NULL;
     }
 
+  /* Use a temporary variable because we don't want
+   * muc_channel_closed_cb or tubes_channel_closed_cb to remove the channel
+   * from the hash table a second time */
   if (priv->text_channels != NULL)
     {
       GHashTable *tmp = priv->text_channels;
-- 
1.5.6.5



More information about the Telepathy-commits mailing list