diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2019-12-23 13:36:05 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-30 11:59:14 +0100 |
| commit | 8744025545236c4623b04720cf1077293d6d651d (patch) | |
| tree | 3a976d2710fc9f149238ccc8133c39bbbc648461 | |
| parent | a4dcf5dd68f243240930bb57b5586827dc44f933 (diff) | |
iface-modem-simple: fix warnings with -Wimplicit-fallthrough
mm-iface-modem-simple.c: In function ‘connection_step’:
mm-iface-modem-simple.c:539:18: error: this statement may fall through [-Werror=implicit-fallthrough=]
539 | ctx->step++;
| ~~~~~~~~~^~
...
| -rw-r--r-- | src/mm-iface-modem-simple.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/mm-iface-modem-simple.c b/src/mm-iface-modem-simple.c index 7d1f0e9e..ac845d76 100644 --- a/src/mm-iface-modem-simple.c +++ b/src/mm-iface-modem-simple.c @@ -536,8 +536,8 @@ connection_step (ConnectionContext *ctx) switch (ctx->step) { case CONNECTION_STEP_FIRST: - /* Fall down to next step */ ctx->step++; + /* fall through */ case CONNECTION_STEP_UNLOCK_CHECK: mm_info ("Simple connect state (%d/%d): Unlock check", @@ -590,9 +590,9 @@ connection_step (ConnectionContext *ctx) } /* If not 3GPP and not CDMA, this will possibly be a POTS modem, - * which won't require any specific registration anywhere. - * So, fall down to next step */ + * which won't require any specific registration anywhere. */ ctx->step++; + /* fall through */ case CONNECTION_STEP_BEARER: { MMBearerList *list = NULL; @@ -675,9 +675,8 @@ connection_step (ConnectionContext *ctx) mm_base_bearer_get_path (ctx->bearer)); g_object_unref (list); g_object_unref (bearer_properties); - /* Fall down to next step */ ctx->step++; - } + } /* fall through */ case CONNECTION_STEP_CONNECT: mm_info ("Simple connect state (%d/%d): Connect", @@ -699,8 +698,8 @@ connection_step (ConnectionContext *ctx) mm_dbg ("Bearer at '%s' is already connected...", mm_base_bearer_get_path (ctx->bearer)); - /* Fall down to next step */ ctx->step++; + /* fall through */ case CONNECTION_STEP_LAST: mm_info ("Simple connect state (%d/%d): All done", |