diff src/protocols/msn/user.c @ 13693:510b7b7dd645

[gaim-migrate @ 16096] Part of SF Patch #1437836 from Sadrul '3. replace a bunch of "if (a) free(a);" with "free(a);"' My comment in the patch: "I'm accepting this, even though it doesn't really do much. In the future, however, I would suggest avoiding creating these type of changes unless you're already editing the code directly around them." committer: Tailor Script <[e-mail]>

--- a/src/protocols/msn/user.c	Tue Apr 25 17:27:17 2006 +0000
+++ b/src/protocols/msn/user.c	Tue Apr 25 17:38:43 2006 +0000
@@ -64,23 +64,12 @@
 	if (user->msnobj != NULL)
 		msn_object_destroy(user->msnobj);
 
-	if (user->passport != NULL)
-		g_free(user->passport);
-
-	if (user->friendly_name != NULL)
-		g_free(user->friendly_name);
-
-	if (user->store_name != NULL)
-		g_free(user->store_name);
-
-	if (user->phone.home != NULL)
-		g_free(user->phone.home);
-
-	if (user->phone.work != NULL)
-		g_free(user->phone.work);
-
-	if (user->phone.mobile != NULL)
-		g_free(user->phone.mobile);
+	g_free(user->passport);
+	g_free(user->friendly_name);
+	g_free(user->store_name);
+	g_free(user->phone.home);
+	g_free(user->phone.work);
+	g_free(user->phone.mobile);
 
 	g_free(user);
 }
@@ -134,9 +123,7 @@
 {
 	g_return_if_fail(user != NULL);
 
-	if (user->passport != NULL)
-		g_free(user->passport);
-
+	g_free(user->passport);
 	user->passport = g_strdup(passport);
 }
 
@@ -145,9 +132,7 @@
 {
 	g_return_if_fail(user != NULL);
 
-	if (user->friendly_name != NULL)
-		g_free(user->friendly_name);
-
+	g_free(user->friendly_name);
 	user->friendly_name = g_strdup(name);
 }
 
@@ -156,9 +141,7 @@
 {
 	g_return_if_fail(user != NULL);
 
-	if (user->store_name != NULL)
-		g_free(user->store_name);
-
+	g_free(user->store_name);
 	user->store_name = g_strdup(name);
 }