diff options
| author | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2014-01-03 22:18:51 +0100 |
|---|---|---|
| committer | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2014-01-03 23:06:29 +0100 |
| commit | b8d5ca85d9a6e6f42b8dbd6d47cab7bbcffeea90 (patch) | |
| tree | a64f9b23e4f98279712d3d29bd37659996ce4c39 /src/group.c | |
| parent | 268732324fd37d9b001694a9206a66cce6681c20 (diff) | |
| download | ratpoison-b8d5ca85d9a6e6f42b8dbd6d47cab7bbcffeea90.zip | |
if (ptr) free(ptr) is not a good idiom.
Diffstat (limited to 'src/group.c')
| -rw-r--r-- | src/group.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/group.c b/src/group.c index 2bbfead..348ffcc 100644 --- a/src/group.c +++ b/src/group.c @@ -144,8 +144,7 @@ group_new (int number, char *name) void group_free (rp_group *g) { - if (g->name) - free (g->name); + free (g->name); numset_free (g->numset); numset_release (group_numset, g->number); free (g); @@ -195,8 +194,7 @@ group_resort_group (rp_group *g) void group_rename (rp_group *g, char *name) { - if (g->name) - free (g->name); + free (g->name); g->name = xstrdup (name); } |