From 099a45cf2ed279902ad896e32fbe451880c61c33 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 12 Apr 2026 15:19:27 -1000 Subject: [PATCH] bot comments --- esphome/components/ld24xx/ld24xx.h | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/esphome/components/ld24xx/ld24xx.h b/esphome/components/ld24xx/ld24xx.h index f919bd1c33..cba1b68a15 100644 --- a/esphome/components/ld24xx/ld24xx.h +++ b/esphome/components/ld24xx/ld24xx.h @@ -22,15 +22,9 @@ LOG_SENSOR(tag, name, (sensor).get_sensor()); \ } -#define SAFE_PUBLISH_SENSOR(sensor, value) \ - if (true) { \ - (sensor).publish_state_if_not_dup(value); \ - } +#define SAFE_PUBLISH_SENSOR(sensor, value) (sensor).publish_state_if_not_dup(value) -#define SAFE_PUBLISH_SENSOR_UNKNOWN(sensor) \ - if (true) { \ - (sensor).publish_state_unknown(); \ - } +#define SAFE_PUBLISH_SENSOR_UNKNOWN(sensor) (sensor).publish_state_unknown() #define highbyte(val) (uint8_t)((val) >> 8) #define lowbyte(val) (uint8_t)((val) &0xff) @@ -72,7 +66,10 @@ inline void format_version_str(const uint8_t *version, std::span buffe /// Stored inline, no heap allocation. Does nothing when no sensor is set. template class SensorWithDedup { public: - void set_sensor(sensor::Sensor *sens) { this->sens_ = sens; } + void set_sensor(sensor::Sensor *sens) { + this->sens_ = sens; + this->dedup_ = {}; + } void publish_state_if_not_dup(T state) { if (this->sens_ != nullptr && this->dedup_.next(state)) {