nxcomp/src/Log.cpp: don't confuse users with useless warning messages in case…

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: don't confuse users with useless warning messages in case…
8036f691 · Mihai Moldovan · 56dbc215 · 8036f691
Commit 8036f691 authored Dec 27, 2017 by Mihai Moldovan
Hide whitespace changes
Inline Side-by-side

Showing with 12 additions and 8 deletions

Log.cpp nxcomp/src/Log.cpp +12 -8

No files found.
--- 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. */
+            /* Discard, if the message is not supposed to be written out anyway. */
-            if (out.has_buffer())
+            if (out.will_log ())
            {
-                out << " (cont.) ";
+                /* And the buffer must of course be non-empty. */
-            }
+                if (out.has_buffer())
-            else
+                {
-            {
+                    out << " (cont.) ";
-                std::cerr << "WARNING: Append operation requested, but no queued data available. "
+                }
-                          << "Internal state error!\n" << "Log line will be discarded!" << std::endl;
+                else
+                {
+                    std::cerr << "WARNING: Append operation requested, but no queued data available. "
+                              << "Internal state error!\n" << "Log line will be discarded!" << std::endl;
+                }
            }
        }
        else