From eb63ea9d377df032485b2bff16ae970dead6f4ab Mon Sep 17 00:00:00 2001 From: Martin Pulec Date: Wed, 12 Apr 2023 12:33:28 +0200 Subject: [PATCH] log_vprintf: append TERM_RESET only if needed It hasn't hurt but it is an extra work and there is no need to issue clearing esc if no formatting was changed. --- src/debug.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/debug.cpp b/src/debug.cpp index 83d04f86b..4a0e7a66d 100644 --- a/src/debug.cpp +++ b/src/debug.cpp @@ -112,15 +112,15 @@ void log_vprintf(int level, const char *format, va_list ap) auto & str = buf.get(); str.resize(str.size() - 1); // drop '\0' written by vsnprintf - if (get_log_output().is_interactive()) { + if (!get_log_output().is_interactive()) { + prune_ansi_sequences_inplace(buf.get()); + } else if (style.size() > 0) { if (str.at(str.size() - 1) == '\n') { // put TERM_RESET before '\n' str.erase(str.size() - 1); buf.append(TERM_RESET "\n"); } else { buf.append(TERM_RESET); } - } else { - prune_ansi_sequences_inplace(buf.get()); } buf.submit(); }