From 8036f6914ddb38cd06b27739a84b3a3667413c14 Mon Sep 17 00:00:00 2001 From: Mihai Moldovan Date: Wed, 27 Dec 2017 20:12:26 +0100 Subject: nxcomp/src/Log.cpp: don't confuse users with useless warning messages in case that a log line append operation was requested, but the message level has lower precedence than the current log filter. --- nxcomp/src/Log.cpp | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) (limited to 'nxcomp') diff --git a/nxcomp/src/Log.cpp b/nxcomp/src/Log.cpp index a3a7222ef..4a9a2679d 100644 --- a/nxcomp/src/Log.cpp +++ b/nxcomp/src/Log.cpp @@ -123,15 +123,19 @@ NXLog& operator<< (NXLog& out, const NXLogStamp& value) /* Appending means that the log object's internal level and the message level must match. */ if (out.current_level() == value.level()) { - /* And the buffer must of course be non-empty. */ - if (out.has_buffer()) + /* Discard, if the message is not supposed to be written out anyway. */ + if (out.will_log ()) { - out << " (cont.) "; - } - else - { - std::cerr << "WARNING: Append operation requested, but no queued data available. " - << "Internal state error!\n" << "Log line will be discarded!" << std::endl; + /* And the buffer must of course be non-empty. */ + if (out.has_buffer()) + { + out << " (cont.) "; + } + else + { + std::cerr << "WARNING: Append operation requested, but no queued data available. " + << "Internal state error!\n" << "Log line will be discarded!" << std::endl; + } } } else -- cgit v1.2.3