From ef5fe9b8d5a273ff91b573fc40b223605c618c1e Mon Sep 17 00:00:00 2001
From: Aleksander Morgado <aleksandermj@google.com>
Date: Mon, 3 Feb 2025 10:21:32 +0000
Subject: [PATCH] broadband-modem: prefer explicit return to break

Just to make it explicit that there is no more logic running
afterwards.
---
 src/mm-broadband-modem.c | 10 +++++-----
 1 file 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 ();