AP_Notify: Perform common checks first

ArduPilot · Jun 26, 2024 · fa5327f · fa5327f
1 parent c388943
commit fa5327f
Showing 1 changed file with 7 additions and 10 deletions.
diff --git a/libraries/AP_Notify/RGBLed.cpp b/libraries/AP_Notify/RGBLed.cpp
@@ -165,16 +165,13 @@ uint32_t RGBLed::get_colour_sequence(void) const
     if (!AP_Notify::flags.pre_arm_check) {
         return sequence_prearm_failing;
     }
-    if (AP_Notify::flags.gps_status >= AP_GPS::GPS_OK_FIX_3D_DGPS &&
-        AP_Notify::flags.pre_arm_gps_check &&
-        good_ahrs_location) {
-        return sequence_disarmed_good_dgps_and_location;
-    }
-
-    if (AP_Notify::flags.gps_status >= AP_GPS::GPS_OK_FIX_3D &&
-        AP_Notify::flags.pre_arm_gps_check &&
-        good_ahrs_location) {
-        return sequence_disarmed_good_gps_and_location;
+    if (AP_Notify::flags.pre_arm_gps_check && good_ahrs_location) {
+        if (AP_Notify::flags.gps_status >= AP_GPS::GPS_OK_FIX_3D_DGPS) {
+            return sequence_disarmed_good_dgps_and_location;
+        }
+        if (AP_Notify::flags.gps_status >= AP_GPS::GPS_OK_FIX_3D) {
+            return sequence_disarmed_good_gps_and_location;
+        }
     }
 #endif