From a451491b476def15acfc3a89605b0793ac6d1103 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Wed, 11 Feb 2026 08:46:30 -0600 Subject: [PATCH] tweak semantics to match --- esphome/components/logger/task_log_buffer_zephyr.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/esphome/components/logger/task_log_buffer_zephyr.cpp b/esphome/components/logger/task_log_buffer_zephyr.cpp index 71c75231b72..44d12d08a3b 100644 --- a/esphome/components/logger/task_log_buffer_zephyr.cpp +++ b/esphome/components/logger/task_log_buffer_zephyr.cpp @@ -61,8 +61,8 @@ bool TaskLogBuffer::send_message_thread_safe(uint8_t level, const char *tag, uin char *text_area = msg->text_data(); ret = vsnprintf(text_area, text_length + 1, format, args); - // Handle unexpected formatting error - if (ret <= 0) { + // Handle unexpected formatting error (ret < 0 is encoding error; ret == 0 is valid empty output) + if (ret < 0) { // this should not happen, vsnprintf was called already once // fill with '\n' to not call mpsc_pbuf_free from producer // it will be trimmed anyway