SensESP
SensESP copied to clipboard
Chain Counter Example uses DigitalInputChange() incorrectly
I noticed that the Chain Counter example uses the DigitalInputChange() method incorrectly. The method definition in https://github.com/SignalK/SensESP/blob/main/src/sensesp/sensors/digital_input.h has the argument list as being
DigitalInputChange(uint8_t pin, int pin_mode, int interrupt_type, String config_path = "")
whereas the usage in the example https://github.com/SignalK/SensESP/blob/main/examples/chain_counter.cpp is
auto* button_watcher = new DigitalInputChange( BUTTON_PIN, INPUT, read_delay, read_delay_config_path);
The error is that the example passes a read_delay value where the method expects an interrupt_type. Both values are ints, so the compiler doesn't notice the problem.