diff options
| author | Simon McVittie <smcv@debian.org> | 2016-10-13 23:09:34 +0100 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2017-01-17 17:20:53 +0000 |
| commit | 21eb3317c569cfd02d799aa67ae3ecf611373cab (patch) | |
| tree | 01553d0e8c3fa700ae7d3dd5a80e8b6c8e55b756 /bus/config-parser-common.c | |
| parent | 81a99b483349313db44d3add31269a424fec1b9b (diff) | |
| download | dbus-21eb3317c569cfd02d799aa67ae3ecf611373cab.tar.gz | |
Move defaults for some switches into a default case
This is clearly equivalent, and quiets -Wswitch-default.
Based on part of a patch by Thomas Zimmermann.
Signed-off-by: Simon McVittie <smcv@debian.org>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=98191
Diffstat (limited to 'bus/config-parser-common.c')
| -rw-r--r-- | bus/config-parser-common.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/bus/config-parser-common.c b/bus/config-parser-common.c index 5db6b289..c1c4191b 100644 --- a/bus/config-parser-common.c +++ b/bus/config-parser-common.c @@ -187,10 +187,9 @@ bus_config_parser_element_type_to_name (ElementType type) return "allow_anonymous"; case ELEMENT_APPARMOR: return "apparmor"; + default: + _dbus_assert_not_reached ("bad element type"); + return NULL; } - - _dbus_assert_not_reached ("bad element type"); - - return NULL; } |