Mercurial > pidgin
comparison libpurple/protocols/msn/group.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 | 723b5a2f91ce |
children |
comparison
equal
deleted
inserted
replaced
20990:bc731b26335c | 20991:c8f99cb61dc9 |
---|---|
56 msn_group_set_id(MsnGroup *group, const char *id) | 56 msn_group_set_id(MsnGroup *group, const char *id) |
57 { | 57 { |
58 g_return_if_fail(group != NULL); | 58 g_return_if_fail(group != NULL); |
59 g_return_if_fail(id != NULL); | 59 g_return_if_fail(id != NULL); |
60 | 60 |
61 g_free(group->id); | |
61 group->id = g_strdup(id); | 62 group->id = g_strdup(id); |
62 } | 63 } |
63 | 64 |
64 void | 65 void |
65 msn_group_set_name(MsnGroup *group, const char *name) | 66 msn_group_set_name(MsnGroup *group, const char *name) |
66 { | 67 { |
67 g_return_if_fail(group != NULL); | 68 g_return_if_fail(group != NULL); |
68 g_return_if_fail(name != NULL); | 69 g_return_if_fail(name != NULL); |
69 | 70 |
70 if (group->name != NULL) | 71 g_free(group->name); |
71 g_free(group->name); | |
72 | |
73 group->name = g_strdup(name); | 72 group->name = g_strdup(name); |
74 } | 73 } |
75 | 74 |
76 char* | 75 char* |
77 msn_group_get_id(const MsnGroup *group) | 76 msn_group_get_id(const MsnGroup *group) |