mirror/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-02-25 21:43:14 -07:00
Code Issues Packages Projects Releases Wiki Activity

review

Browse Source
This commit is contained in:
J. Nick Koston
2026-01-14 15:56:32 -10:00
parent e13743a9c3
commit 4befd86a96
1 changed files with 27 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
esphome/components/cse7766/cse7766.cpp
View File

@@ -2,6 +2,7 @@
#include "esphome/core/application.h"
#include "esphome/core/helpers.h"
#include "esphome/core/log.h"
#include <cstdarg>
namespace esphome {
namespace cse7766 {
@@ -9,6 +10,25 @@ namespace cse7766 {
static const char *const TAG = "cse7766";
static constexpr size_t CSE7766_RAW_DATA_SIZE = 24;
#if ESPHOME_LOG_LEVEL >= ESPHOME_LOG_LEVEL_VERY_VERBOSE
/// Safely append formatted string to buffer, returning new position (capped at size).
/// Handles negative return values from snprintf (encoding errors).
__attribute__((format(printf, 4, 5))) static size_t buf_append_(char *buf, size_t size, size_t pos, const char *fmt,
...) {
if (pos >= size) {
return size;
}
va_list args;
va_start(args, fmt);
int written = vsnprintf(buf + pos, size - pos, fmt, args);
va_end(args);
if (written < 0) {
return pos; // encoding error
}
return std::min(pos + static_cast<size_t>(written), size);
}
#endif
void CSE7766Component::loop() {
const uint32_t now = App.get_loop_component_start_time();
if (now - this->last_transmission_ >= 500) {
@@ -207,23 +227,21 @@ void CSE7766Component::parse_data_() {
#if ESPHOME_LOG_LEVEL >= ESPHOME_LOG_LEVEL_VERY_VERBOSE
{
// Buffer: 7 + 14 + 31 + 14 + 24 = 90 chars max + null, rounded to 96
// Buffer: 7 + 14 + 31 + 14 + 24 = 90 chars max + null, rounded to 96.
// Float sizes assume typical sensor values (voltage ~220V, current <10A, power <3000W).
char buf[96];
int pos = snprintf(buf, sizeof(buf), "Parsed:"); // max 7: "Parsed:"
size_t pos = buf_append_(buf, sizeof(buf), 0, "Parsed:");
if (have_voltage) {
pos += snprintf(buf + pos, sizeof(buf) - pos, " V=%.4fV", voltage); // max 14: " V="(3) + float(10) + "V"(1)
pos = buf_append_(buf, sizeof(buf), pos, " V=%.4fV", voltage);
}
if (have_current) {
pos +=
snprintf(buf + pos, sizeof(buf) - pos, " I=%.4fmA (~%.4fmA)", current * 1000.0f,
calculated_current * 1000.0f); // max 31: " I="(3) + float(10) + "mA (~"(5) + float(10) + "mA)"(3)
pos = buf_append_(buf, sizeof(buf), pos, " I=%.4fmA (~%.4fmA)", current * 1000.0f, calculated_current * 1000.0f);
}
if (have_power) {
pos += snprintf(buf + pos, sizeof(buf) - pos, " P=%.4fW", power); // max 14: " P="(3) + float(10) + "W"(1)
pos = buf_append_(buf, sizeof(buf), pos, " P=%.4fW", power);
}
if (energy != 0.0f) {
snprintf(buf + pos, sizeof(buf) - pos, " E=%.4fkWh (%u)", energy,
cf_pulses); // max 24: " E="(3) + float(10) + "kWh ("(5) + uint16(5) + ")"(1)
buf_append_(buf, sizeof(buf), pos, " E=%.4fkWh (%u)", energy, cf_pulses);
}
ESP_LOGVV(TAG, "%s", buf);
}