Swap Previous Period helper functions to ulong

Author: tyeth

platformio.ini

@@ -275,6 +275,7 @@ extra_scripts =  pre:rename_usb_config.py
 [env:adafruit_metro_esp32s3_debug]
 extends = common:esp32
 board = adafruit_metro_esp32s3
+build_type = debug
 build_flags = -DARDUINO_METRO_ESP32S3 -DBOARD_HAS_PSRAM -DCFG_TUSB_DEBUG=1 -DDEBUG=1 -DESP_LOG_LEVEL=ESP_LOG_VERBOSE -DARDUINO_CORE_DEBUG_LEVEL=5 -DCORE_DEBUG_LEVEL=5 -DARDUHAL_LOG_LEVEL=5 -DARDUINO_USB_CDC_ON_BOOT=1
 ;set partition to tinyuf2-partitions-16MB.csv as of idf 5.1
 board_build.partitions = tinyuf2-partitions-16MB.csv

src/components/i2c/WipperSnapper_I2C.cpp

@@ -1328,16 +1328,16 @@ void WipperSnapper_Component_I2C::sensorEventRead(
     unsigned long curTime, wippersnapper_signal_v1_I2CResponse *msgi2cResponse,
     bool (WipperSnapper_I2C_Driver::*getEventFunc)(sensors_event_t *),
     long (WipperSnapper_I2C_Driver::*getPeriodFunc)(),
-    long (WipperSnapper_I2C_Driver::*getPeriodPrvFunc)(),
-    void (WipperSnapper_I2C_Driver::*setPeriodPrvFunc)(long),
+    ulong (WipperSnapper_I2C_Driver::*getPeriodPrvFunc)(),
+    void (WipperSnapper_I2C_Driver::*setPeriodPrvFunc)(ulong),
     wippersnapper_i2c_v1_SensorType sensorType, const char *sensorName,
     const char *unit, sensors_event_t event,
     float sensors_event_t::*valueMember, bool &sensorsReturningFalse,
     int &retries) {
   // sensorName used for prefix + error message, units is value suffix
   curTime = millis();
   if (((*iter)->*getPeriodFunc)() != 0L &&
-      curTime - ((*iter)->*getPeriodPrvFunc)() > ((*iter)->*getPeriodFunc)()) {
+      curTime - ((*iter)->*getPeriodPrvFunc)() > (ulong)((*iter)->*getPeriodFunc)()) {
     // within the period, read the sensor
     if (((*iter)->*getEventFunc)(&event)) {
       float value;

src/components/i2c/WipperSnapper_I2C.h

@@ -106,8 +106,8 @@ class WipperSnapper_Component_I2C {
       wippersnapper_signal_v1_I2CResponse *msgi2cResponse,
       bool (WipperSnapper_I2C_Driver::*getEventFunc)(sensors_event_t *),
       long (WipperSnapper_I2C_Driver::*getPeriodFunc)(),
-      long (WipperSnapper_I2C_Driver::*getPeriodPrvFunc)(),
-      void (WipperSnapper_I2C_Driver::*setPeriodPrvFunc)(long),
+      ulong (WipperSnapper_I2C_Driver::*getPeriodPrvFunc)(),
+      void (WipperSnapper_I2C_Driver::*setPeriodPrvFunc)(ulong),
       wippersnapper_i2c_v1_SensorType sensorType, const char *sensorName,
       const char *unit, sensors_event_t event,
       float sensors_event_t::*valueMember, bool &sensorsReturningFalse,