Changeset 511 for cpp/common/loggers/loggers.cpp

Timestamp:

05/23/16 13:48:45 (7 years ago)

Author:

Maciej Komosinski

Message:

Improved handling of multiline messages by loggers

File:

• cpp/common/loggers/loggers.cpp (modified) (2 diffs)

cpp/common/loggers/loggers.cpp

@@ -5 +5 @@
 #include "loggers.h"
 #include <common/stl-util.h>
+#include <string.h>
 
-void _logMessageSingleLine(const char *obj, const char *method, int level, const char *msg)
+void logMessage(const char *obj, const char *method, int level, const char *msg)
 {
         tlsGetRef(message_handler_manager_instance).send(obj, method, level, msg);
@@ -101 +102 @@
 }
 
+void LoggerBase::handle(const char *obj, const char *method, int level, const char *msg)
+{
+        int line = 0; //all lines except the first one get the "..." prefix
+        const char* nextsep;
+        do
+        {
+                nextsep = strchr(msg, '\n');
+                if (nextsep == NULL) //last chunk, until the end
+                        nextsep = strchr(msg, '\0');
+                if ((nextsep > msg) && (nextsep[-1] == '\r'))
+                        nextsep--;
+                if (line == 0)
+                {
+                        if (*nextsep == 0) //there was only one line! no need to modify it in any way.
+                                handleSingleLine(obj, method, level, msg);
+                        else //first line from multi-line
+                                handleSingleLine(obj, method, level, string(msg, nextsep - msg).c_str());
+                }
+                else //consecutive lines from multi-line
+                        handleSingleLine(obj, method, level, (LOG_MULTILINE_CONTINUATION + string(msg, nextsep - msg)).c_str()); //could also add line numbers like ...(3)... but let's keep the prefix short and simple
+                line++;
+                if ((nextsep[0] == '\r') && (nextsep[1] == '\n'))
+                        msg = nextsep + 2;
+                else if (*nextsep)
+                        msg = nextsep + 1;
+        } while (*nextsep);
+}
+
 /////////////////////////////////