Changeset 244 for cpp/common

Timestamp:

05/31/14 21:31:01 (11 years ago)

Author:

Maciej Komosinski

Message:

Printing multi-line messages splits the message into separate lines and adds "..." prefix

Location:

cpp/common

Files:

• framsg.cpp (modified) (1 diff)
• framsg.h (modified) (1 diff)
• nonstd.h (modified) (1 diff)

cpp/common/framsg.cpp

@@ -9 +9 @@
 
 const char* MSG_LEVEL[]={"DEBUG","INFO","WARN","ERROR","CRITICAL"};
+
+void FramMessage(const char *o, const char *m, const char *txt, int w)
+{
+        int line = 0; //all lines except the first one get the "..." prefix
+        const char* nextsep;
+        do
+        {
+                nextsep = strchr(txt, '\n');
+                if (nextsep == NULL) //last chunk, until the end
+                        nextsep = strchr(txt, '\0');
+                if ((nextsep > txt) && (nextsep[-1] == '\r'))
+                        nextsep--;
+                if (line == 0)
+                {
+                        if (*nextsep == 0) //there was only one line! no need to modify it in any way.
+                                _FramMessageSingleLine(o, m, txt, w);
+                        else //first line from multi-line
+                                _FramMessageSingleLine(o, m, string(txt, nextsep - txt).c_str(), w);
+                }
+                else //consecutive lines from multi-line
+                        _FramMessageSingleLine(o, m, ("..." + string(txt, nextsep - txt)).c_str(), w); //could also add line numbers like ...(3)... but let's keep the prefix short and simple
+                line++;
+                if ((nextsep[0] == '\r') && (nextsep[1] == '\n'))
+                        txt = nextsep + 2;
+                else if (*nextsep)
+                        txt = nextsep + 1;
+        } while (*nextsep);
+}
+
 
 void FMprintf_va(const char *o,const char *m,int w,const char *bl,va_list va)

cpp/common/framsg.h

@@ -13 +13 @@
 void FMprintf_va(const char *o,const char *m,int w,const char *bl,va_list va); //a different name than FMprintf - otherwise the compiler could confuse the "string" parameter with va_list and could call the wrong function
 void printFM(const char *bl,...); //a shorthand for printf (a different name again to avoid the risk of confusion with the two functions above. This would be unlikely but possible when the argument types would match)
-void FramMessage(const char *o,const char *m,const char *bl,int w);
+void FramMessage(const char *o,const char *m,const char *txt,int w);
+
+void _FramMessageSingleLine(const char *o,const char *m,const char *txt,int w); //don't call this directly - it is used internally
 
 #define FMLV_DEBUG -1

cpp/common/nonstd.h

@@ -57 +57 @@
 #endif
 
-#ifdef MACOS
+#if defined MACOS || defined __ANDROID__
  #define stricmp(a,b) strcasecmp(a,b)
 #endif