Mercurial > pidgin.yaz
diff libpurple/protocols/msn/user.c @ 20991:c8f99cb61dc9
Cleanup some unnecessary NULL checks in g_strdup()/g_free().
| author | Daniel Atallah <daniel.atallah@gmail.com> |
|---|---|
| date | Fri, 19 Oct 2007 01:38:35 +0000 |
| parents | 0cbfc19e4909 |
| children | 6de09629f091 |
line wrap: on
line diff
--- a/libpurple/protocols/msn/user.c Fri Oct 19 01:28:55 2007 +0000 +++ b/libpurple/protocols/msn/user.c Fri Oct 19 01:38:35 2007 +0000 @@ -391,10 +391,8 @@ { g_return_if_fail(user != NULL); - if (user->phone.home != NULL) - g_free(user->phone.home); - - user->phone.home = (number == NULL ? NULL : g_strdup(number)); + g_free(user->phone.home); + user->phone.home = g_strdup(number); } void @@ -402,10 +400,8 @@ { g_return_if_fail(user != NULL); - if (user->phone.work != NULL) - g_free(user->phone.work); - - user->phone.work = (number == NULL ? NULL : g_strdup(number)); + g_free(user->phone.work); + user->phone.work = g_strdup(number); } void @@ -413,10 +409,8 @@ { g_return_if_fail(user != NULL); - if (user->phone.mobile != NULL) - g_free(user->phone.mobile); - - user->phone.mobile = (number == NULL ? NULL : g_strdup(number)); + g_free(user->phone.mobile); + user->phone.mobile = g_strdup(number); } void