diff options
author | Marcel Holtmann <marcel@holtmann.org> | 2010-03-20 02:26:57 +0100 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2010-03-20 02:26:57 +0100 |
commit | 256c6e14c7947a10833cfcacf2ee74422d0d7563 (patch) | |
tree | c38f73fa317a9c4d3dfe4af300f79cf573273366 /src/common.c | |
parent | d6e78f78287599c94dde744e1b856b27f6d1b884 (diff) | |
download | ofono-256c6e14c7947a10833cfcacf2ee74422d0d7563.tar.bz2 |
Revert the default statement for PIN type checking
Also fix the coding style for the switch statement and remove the
number assignment for an internal enum.
Diffstat (limited to 'src/common.c')
-rw-r--r-- | src/common.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/common.c b/src/common.c index 32b64d49..b58d9bee 100644 --- a/src/common.c +++ b/src/common.c @@ -592,8 +592,7 @@ gboolean is_valid_pin(const char *pin, enum pin_type type) if (i != strspn(pin, "012345679")) return FALSE; - switch (type) - { + switch (type) { case PIN_TYPE_PIN: /* 11.11 Section 9.3 ("CHV"): 4..8 IA-5 digits */ if (4 <= i && i <= 8) @@ -613,8 +612,6 @@ gboolean is_valid_pin(const char *pin, enum pin_type type) if (i < 8) return TRUE; break; - default: - break; } return FALSE; |