diff options
| author |   millert <millert@openbsd.org> | 2015-04-01 21:16:17 +0000 |
|---|---|---|
| committer | millert <millert@openbsd.org> | 2015-04-01 21:16:17 +0000 |
| commit | ed004aa2319f73a408e885987aadec409cb52fd1 (patch) | |
| tree | dfbc95c26e5e90778599cb4d45a8070235d2acfe /usr.bin/sort/sort.c | |
| parent | No need for an else clause after a continue, it just makes the (diff) | |
| download | wireguard-openbsd-ed004aa2319f73a408e885987aadec409cb52fd1.tar.xz wireguard-openbsd-ed004aa2319f73a408e885987aadec409cb52fd1.zip | |
No need for if (x) free(x) style checks, this is not K&R.
Diffstat (limited to 'usr.bin/sort/sort.c')
| -rw-r--r-- | usr.bin/sort/sort.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/usr.bin/sort/sort.c b/usr.bin/sort/sort.c index 7da8b00ab28..74dd3f64aa7 100644 --- a/usr.bin/sort/sort.c +++ b/usr.bin/sort/sort.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sort.c,v 1.60 2015/04/01 21:13:07 millert Exp $ */ +/* $OpenBSD: sort.c,v 1.61 2015/04/01 21:16:17 millert Exp $ */ /*- * Copyright (C) 2009 Gabor Kovesdan <gabor@FreeBSD.org> @@ -619,10 +619,8 @@ parse_pos(const char *s, struct key_specs *ks, bool *mef_flags, bool second) end: - if (c) - sort_free(c); - if (f) - sort_free(f); + sort_free(c); + sort_free(f); regfree(&re); return ret; @@ -739,10 +737,8 @@ parse_pos_obs(const char *s, size_t *nf, size_t *nc, char *sopts, ret = 0; end: - if (c) - sort_free(c); - if (f) - sort_free(f); + sort_free(c); + sort_free(f); regfree(&re); return ret; |