diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-11 13:57:08 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-31 14:52:49 +0100 |
| commit | 5eedad988ee626a08b275bff021feedffdedc66e (patch) | |
| tree | 798640e67dddab2cf4f0c6d85aee651ae00a1180 /src | |
| parent | 6dfb9d7a7e1f53e5aac3f28b2f6a249db0dd2b4c (diff) | |
sms-part-3gpp: fix warnings with -Wswitch-default
mm-sms-part-3gpp.c: In function ‘sms_encoding_type’:
mm-sms-part-3gpp.c:190:13: error: switch missing default case [-Werror=switch-default]
190 | switch (dcs & 0x0c) {
| ^~~~~~
mm-sms-part-3gpp.c:219:13: error: switch missing default case [-Werror=switch-default]
219 | switch (dcs & 0x04) {
| ^~~~~~
Diffstat (limited to 'src')
| -rw-r--r-- | src/mm-sms-part-3gpp.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/mm-sms-part-3gpp.c b/src/mm-sms-part-3gpp.c index d9779898..725a35fb 100644 --- a/src/mm-sms-part-3gpp.c +++ b/src/mm-sms-part-3gpp.c @@ -192,14 +192,16 @@ sms_encoding_type (int dcs) scheme = MM_SMS_ENCODING_UCS2; break; case 0x00: - /* fallthrough */ /* reserved - spec says to treat it as default alphabet */ + /* Fall through */ case 0x0c: scheme = MM_SMS_ENCODING_GSM7; break; case 0x04: scheme = MM_SMS_ENCODING_8BIT; break; + default: + g_assert_not_reached (); } break; @@ -223,6 +225,8 @@ sms_encoding_type (int dcs) case 0x04: scheme = MM_SMS_ENCODING_8BIT; break; + default: + g_assert_not_reached (); } break; @@ -689,6 +693,8 @@ mm_sms_part_3gpp_new_from_binary_pdu (guint index, mm_sms_part_set_concat_max (sms_part,pdu[offset + 2]); mm_sms_part_set_concat_sequence (sms_part, pdu[offset + 3]); break; + default: + break; } offset += ie_len; |