diff options
author | Marcel Holtmann <marcel@holtmann.org> | 2009-01-01 19:46:14 +0100 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2009-01-01 19:46:14 +0100 |
commit | f89b202b68eaf920e969e155aee2f40f5b063c45 (patch) | |
tree | d32340a21fe4eb3f07f949258b93f0b29531b925 /src/detect.c | |
parent | 98ffc630a2e7a982174f698fc99c6c905e4896a9 (diff) | |
download | connman-f89b202b68eaf920e969e155aee2f40f5b063c45.tar.gz connman-f89b202b68eaf920e969e155aee2f40f5b063c45.tar.bz2 connman-f89b202b68eaf920e969e155aee2f40f5b063c45.zip |
Don't use default for enum based switch statements
Diffstat (limited to 'src/detect.c')
-rw-r--r-- | src/detect.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/detect.c b/src/detect.c index b82974ab..8d64aeba 100644 --- a/src/detect.c +++ b/src/detect.c @@ -177,18 +177,20 @@ static void detect_newlink(unsigned short type, int index, devtype = CONNMAN_DEVICE_TYPE_HSO; } - if (devtype == CONNMAN_DEVICE_TYPE_UNKNOWN) { + switch (devtype) { + case CONNMAN_DEVICE_TYPE_UNKNOWN: g_free(devname); return; - } - - switch (devtype) { + case CONNMAN_DEVICE_TYPE_ETHERNET: + case CONNMAN_DEVICE_TYPE_WIFI: + case CONNMAN_DEVICE_TYPE_WIMAX: + name = index2ident(index, "dev_"); + break; + case CONNMAN_DEVICE_TYPE_BLUETOOTH: case CONNMAN_DEVICE_TYPE_HSO: + case CONNMAN_DEVICE_TYPE_VENDOR: name = strdup(devname); break; - default: - name = index2ident(index, "dev_"); - break; } device = connman_device_create(name, devtype); |