diff options
| author | Aleksander Morgado <aleksandermj@google.com> | 2025-02-03 10:21:32 +0000 |
|---|---|---|
| committer | Aleksander Morgado <aleksandermj@google.com> | 2025-02-03 10:21:32 +0000 |
| commit | ef5fe9b8d5a273ff91b573fc40b223605c618c1e (patch) | |
| tree | 7d6332b670e7f825f560bf005ff3805fe3d0e834 | |
| parent | 4ce23a9ebd02b84d1dcc1b98f0bd0d0f7637d039 (diff) | |
broadband-modem: prefer explicit return to break
Just to make it explicit that there is no more logic running
afterwards.
| -rw-r--r-- | src/mm-broadband-modem.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/mm-broadband-modem.c b/src/mm-broadband-modem.c index 7548f223..c4140015 100644 --- a/src/mm-broadband-modem.c +++ b/src/mm-broadband-modem.c @@ -12666,19 +12666,19 @@ enable (MMBaseModem *self, g_task_return_new_error (task, MM_CORE_ERROR, MM_CORE_ERROR_WRONG_STATE, "Cannot enable modem: unknown state"); g_object_unref (task); - break; + return; case MM_MODEM_STATE_FAILED: g_task_return_new_error (task, MM_CORE_ERROR, MM_CORE_ERROR_WRONG_STATE, "Cannot enable modem: initialization failed"); g_object_unref (task); - break; + return; case MM_MODEM_STATE_LOCKED: g_task_return_new_error (task, MM_CORE_ERROR, MM_CORE_ERROR_WRONG_STATE, "Cannot enable modem: device locked"); g_object_unref (task); - break; + return; case MM_MODEM_STATE_INITIALIZING: case MM_MODEM_STATE_DISABLED: @@ -12688,7 +12688,7 @@ enable (MMBaseModem *self, case MM_MODEM_STATE_ENABLING: g_assert_not_reached (); - break; + return; case MM_MODEM_STATE_ENABLED: case MM_MODEM_STATE_SEARCHING: @@ -12699,7 +12699,7 @@ enable (MMBaseModem *self, /* Just return success, don't relaunch enabling */ g_task_return_boolean (task, TRUE); g_object_unref (task); - break; + return; default: g_assert_not_reached (); |