Mercurial > pidgin

diff libpurple/protocols/msn/switchboard.h @ 24395:9b253ce969d0

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
msn_slplink_remove_slpcall() shouldn't be responsible for setting slpcall->slplink to NULL. In fact, msn_slp_call_destroy() shouldn't even call msn_slplink_remove_slpcall() until it knows that it doesn't need the slplink anymore. So move the call to msn_slplink_remove_slpcall() further down in that function. Now we no longer need the return value from msn_switchboard_release(). That return value, by the way, was dumb. If you're no longer using the MsnSwitchBoard then you're not using it and you shouldn't care if it was destroyed or not.
author Mark Doliner <mark@kingant.net>
date Thu, 13 Nov 2008 07:22:39 +0000
parents fb722b8b4c74
children a3523b143f78
line wrap: on
line diff
--- a/libpurple/protocols/msn/switchboard.h	Thu Nov 13 06:57:20 2008 +0000
+++ b/libpurple/protocols/msn/switchboard.h	Thu Nov 13 07:22:39 2008 +0000
@@ -224,10 +224,8 @@
  *
  * @param swboard The switchboard to release.
  * @param flag The flag that states the function.
- *
- * @return @c TRUE if the switchboard was closed, @c FALSE otherwise.
  */
-gboolean msn_switchboard_release(MsnSwitchBoard *swboard, MsnSBFlag flag);
+void msn_switchboard_release(MsnSwitchBoard *swboard, MsnSBFlag flag);
 
 /**
  * Returns whether or not we currently can send a message through this