CLang-format analogIO

Author: tyeth

src/components/analogIO/Wippersnapper_AnalogIO.cpp

@@ -87,7 +87,8 @@ void Wippersnapper_AnalogIO::setADCResolution(int resolution) {
   analogReadResolution(16);
   _nativeResolution = 12;
 #elif defined(ARDUINO_ARCH_ESP32)
-  scaleAnalogRead = false; // should be false, handled in bsp (analogReadResolution)
+  scaleAnalogRead =
+      false; // should be false, handled in bsp (analogReadResolution)
   analogReadResolution(resolution); // 16 bit values (shifted from 12 or 13bit)
 #if defined(ESP32S3)
   _nativeResolution = 13; // S3 ADC is 13-bit, others are 12-bit
@@ -323,12 +324,12 @@ bool Wippersnapper_AnalogIO::encodePinEvent(
     @returns True if pin's period expired, False otherwise.
 */
 /**********************************************************/
-bool Wippersnapper_AnalogIO::timerExpired(long currentTime,
-                                          analogInputPin pin, long periodOffset) {
+bool Wippersnapper_AnalogIO::timerExpired(long currentTime, analogInputPin pin,
+                                          long periodOffset) {
   if (pin.period + periodOffset != 0L &&
-    currentTime - pin.prvPeriod > (pin.period + periodOffset)) {
-      return true;
-    }
+      currentTime - pin.prvPeriod > (pin.period + periodOffset)) {
+    return true;
+  }
   return false;
 }
 
@@ -348,7 +349,7 @@ void Wippersnapper_AnalogIO::update() {
 
       // Does the pin execute on-period?
       if (_analog_input_pins[i].period != 0L &&
-        timerExpired(millis(), _analog_input_pins[i])) {
+          timerExpired(millis(), _analog_input_pins[i])) {
         WS_DEBUG_PRINT("Executing periodic event on A");
         WS_DEBUG_PRINTLN(_analog_input_pins[i].pinName);
 
@@ -376,18 +377,17 @@ void Wippersnapper_AnalogIO::update() {
       // Does the pin execute on_change?
       else if (_analog_input_pins[i].period == 0L) {
 
-        // TODO: after testing combine these two conditionals to just send the event and not debug print.
+        // TODO: after testing combine these two conditionals to just send the
+        // event and not debug print.
         if (_analog_input_pins[i].prvPeriod == 0L) {
-          // first time reading pin, no previous timestamp so send event + update prvPeriod
-          // WS_DEBUG_PRINT("First time reading pin A");
+          // first time reading pin, no previous timestamp so send event +
+          // update prvPeriod WS_DEBUG_PRINT("First time reading pin A");
           // WS_DEBUG_PRINTLN(_analog_input_pins[i].pinName);
-        }
-        else if (timerExpired(millis(), _analog_input_pins[i], 500)) {
+        } else if (timerExpired(millis(), _analog_input_pins[i], 500)) {
           // timer expired, send event + update prvPeriod
           // WS_DEBUG_PRINT("Timer passed for pin A");
           // WS_DEBUG_PRINTLN(_analog_input_pins[i].pinName);
-        }
-        else {
+        } else {
           // last event was too recently, skip this one...
           // WS_DEBUG_PRINTLN("Timer has not expired, continue...");
           continue;
@@ -397,12 +397,14 @@ void Wippersnapper_AnalogIO::update() {
         // pin value
         uint16_t pinValRaw = getPinValue(_analog_input_pins[i].pinName);
 
-        // All boards ADC values scaled to 16bit, in future we may need to adjust dynamically
+        // All boards ADC values scaled to 16bit, in future we may need to
+        // adjust dynamically
         uint16_t maxDecimalValue = 65535;
 
-        // Calculate threshold values - using DEFAULT_HYSTERISIS for first third (1/3) of the range,
-        // then 2x DEFAULT_HYSTERISIS for the middle 1/3, and 4x DEFAULT_HYSTERISIS for the last 1/3.
-        // This should allow a more wifi blip tolerant threshold for the both ends of the range.
+        // Calculate threshold values - using DEFAULT_HYSTERISIS for first third
+        // (1/3) of the range, then 2x DEFAULT_HYSTERISIS for the middle 1/3,
+        // and 4x DEFAULT_HYSTERISIS for the last 1/3. This should allow a more
+        // wifi blip tolerant threshold for the both ends of the range.
         float CURRENT_HYSTERISIS;
         if (pinValRaw < maxDecimalValue / 3) {
           CURRENT_HYSTERISIS = maxDecimalValue * DEFAULT_HYSTERISIS;
@@ -412,15 +414,14 @@ void Wippersnapper_AnalogIO::update() {
           CURRENT_HYSTERISIS = maxDecimalValue * DEFAULT_HYSTERISIS * 4;
         }
 
-
-
         // get the threshold values for previous pin value
         uint16_t _pinValThreshHi =
             _analog_input_pins[i].prvPinVal + CURRENT_HYSTERISIS;
         uint16_t _pinValThreshLow =
             _analog_input_pins[i].prvPinVal - CURRENT_HYSTERISIS;
 
-        if (_analog_input_pins[i].prvPeriod == 0 || pinValRaw > _pinValThreshHi || pinValRaw < _pinValThreshLow) {
+        if (_analog_input_pins[i].prvPeriod == 0 ||
+            pinValRaw > _pinValThreshHi || pinValRaw < _pinValThreshLow) {
           // Perform voltage conversion if we need to
           if (_analog_input_pins[i].readMode ==
               wippersnapper_pin_v1_ConfigurePinRequest_AnalogReadMode_ANALOG_READ_MODE_PIN_VOLTAGE) {