diff options
author | Lennart Poettering <lennart@poettering.net> | 2015-09-19 00:50:34 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2015-09-22 16:31:10 +0200 |
commit | 8c721f2bcb134de228ec98f5e27ce89d79a31b55 (patch) | |
tree | d07f24f87ecac2608ee05bc1d90bcf3664c7273f /src/basic/util.c | |
parent | 08edf879ed3c98e5ae889dceb8b761929f407054 (diff) |
util: clean-ups to enum parsers
Never log when we fail due to OOM when translating enums, let the caller
do that. Translating basic types like enums should be something where
the caller logs, not the translatior functions.
Return -1 when NULL is passed to all enum parser functions.
The non-fallback versions of the enum translator calls already handle
NULL as failure, instead of hitting an assert, and we should do this
here, too.
Diffstat (limited to 'src/basic/util.c')
-rw-r--r-- | src/basic/util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/basic/util.c b/src/basic/util.c index 368b556229..65951f7552 100644 --- a/src/basic/util.c +++ b/src/basic/util.c @@ -6561,7 +6561,7 @@ ssize_t string_table_lookup(const char * const *table, size_t len, const char *k for (i = 0; i < len; ++i) if (streq_ptr(table[i], key)) - return (ssize_t)i; + return (ssize_t) i; return -1; } |