Correct bracketing

Author: PaulZC

Firmware/RTK_Everywhere/GNSS_Mosaic.ino

@@ -390,7 +390,6 @@ bool GNSS_MOSAIC::checkPPPRates()
 //   Returns true if successfully configured and false upon failure
 //----------------------------------------
 // If any of the settings in the signature have changed, reapply the Base configuration
-// See GNSS_MOSAIC_ROVER_SIGNATURE as an example
 #define GNSS_MOSAIC_BASE_SIGNATURE                                      \
     (                                                                   \
         ((settings.minElev & 0x01) << 1)                                \
@@ -418,7 +417,7 @@ bool GNSS_MOSAIC::configureBase()
         return (false);
     }
 
-    if (settings.gnssConfiguredBase == 0x01 | GNSS_MOSAIC_BASE_SIGNATURE)
+    if (settings.gnssConfiguredBase == (0x01 | GNSS_MOSAIC_BASE_SIGNATURE))
     {
         systemPrintln("Skipping mosaic Base configuration");
         setLoggingType(); // Needed because logUpdate exits early and never calls setLoggingType
@@ -572,7 +571,7 @@ bool GNSS_MOSAIC::configureOnce()
     RTCMv3 messages are enabled by enableRTCMRover / enableRTCMBase
     */
 
-    if (settings.gnssConfiguredOnce == 0x01 | GNSS_MOSAIC_ONCE_SIGNATURE)
+    if (settings.gnssConfiguredOnce == (0x01 | GNSS_MOSAIC_ONCE_SIGNATURE))
     {
         systemPrintln("mosaic configuration maintained");
         return (true);
@@ -686,7 +685,7 @@ bool GNSS_MOSAIC::configureRover()
     }
 
     // If our settings haven't changed, trust GNSS's settings
-    if (settings.gnssConfiguredRover == 0x01 | GNSS_MOSAIC_ROVER_SIGNATURE)
+    if (settings.gnssConfiguredRover == (0x01 | GNSS_MOSAIC_ROVER_SIGNATURE))
     {
         systemPrintln("Skipping mosaic Rover configuration");
         setLoggingType(); // Needed because logUpdate exits early and never calls setLoggingType

Firmware/RTK_Everywhere/GNSS_ZED.ino

@@ -333,7 +333,7 @@ bool GNSS_ZED::configureBase()
     if (online.gnss == false)
         return (false);
 
-    if (settings.gnssConfiguredBase == 0x01 | GNSS_ZED_BASE_SIGNATURE)
+    if (settings.gnssConfiguredBase == (0x01 | GNSS_ZED_BASE_SIGNATURE))
     {
         if (settings.debugGnss)
             systemPrintln("Skipping ZED Base configuration");
@@ -593,7 +593,7 @@ bool GNSS_ZED::configureGNSS()
     // Configuring the ZED can take more than 2000ms. Configuration is saved to
     // ZED RAM+BBR so there is no need to update settings unless user has modified
     // the settings file or internal settings.
-    if (settings.gnssConfiguredOnce == 0x01 | GNSS_ZED_ONCE_SIGNATURE)
+    if (settings.gnssConfiguredOnce == (0x01 | GNSS_ZED_ONCE_SIGNATURE))
     {
         systemPrintln("ZED-F9x configuration maintained");
         return (true);
@@ -799,7 +799,7 @@ bool GNSS_ZED::configureRover()
     }
 
     // If our settings haven't changed, trust GNSS's settings
-    if (settings.gnssConfiguredRover == 0x01 | GNSS_ZED_ROVER_SIGNATURE)
+    if (settings.gnssConfiguredRover == (0x01 | GNSS_ZED_ROVER_SIGNATURE))
     {
         systemPrintln("Skipping ZED Rover configuration");
         return (true);