diff libpurple/protocols/msn/command.c @ 31031:61db10475bed

As Mark did before, we probably don't need a return value from these unref functions. Also, make the ref counts unsigned.
author Elliott Sales de Andrade <qulogic@pidgin.im>
date Wed, 15 Dec 2010 04:17:59 +0000
parents 230caecf5435
children 05e05d96ba75
line wrap: on
line diff
--- a/libpurple/protocols/msn/command.c	Wed Dec 15 00:00:12 2010 +0000
+++ b/libpurple/protocols/msn/command.c	Wed Dec 15 04:17:59 2010 +0000
@@ -107,19 +107,17 @@
 	return cmd;
 }
 
-MsnCommand *
+void
 msn_command_unref(MsnCommand *cmd)
 {
-	g_return_val_if_fail(cmd != NULL, NULL);
-	g_return_val_if_fail(cmd->ref_count > 0, NULL);
+	g_return_if_fail(cmd != NULL);
+	g_return_if_fail(cmd->ref_count > 0);
 
 	cmd->ref_count--;
 
 	if (cmd->ref_count == 0)
 	{
 		msn_command_destroy(cmd);
-		return NULL;
 	}
+}
 
-	return cmd;
-}