Changeset 786 for cpp/frams/util

Timestamp:

05/21/18 14:01:14 (7 years ago)

Author:

Maciej Komosinski

Message:

Improved shortening of long stringified values in error messages

Location:

cpp/frams/util

Files:

• extvalue.cpp (modified) (1 diff)
• sstringutils.cpp (modified) (1 diff)
• sstringutils.h (modified) (1 diff)

cpp/frams/util/extvalue.cpp

@@ -305 +305 @@
 {
         SString msg = typeDescription();
+        SString delimit("'");
         switch (type)
         {
-        case TInt: case TDouble: case TString: case TObj:
-                msg += " '";
-                msg += sstringShorten(getString(), 30);
-                msg += "'";
+        case TString:
+                delimit = "\"";
+        case TInt: case TDouble: case TObj:
+                msg += " ";
+                msg += sstringDelimitAndShorten(getString(), 50, (type == TString), delimit, delimit);
         default:;
         }

cpp/frams/util/sstringutils.cpp

@@ -157 +157 @@
 }
 
-SString sstringShorten(const SString &in, int maxlen)
+SString sstringDelimitAndShorten(const SString &in, int maxlen, bool show_length, const SString& before, const SString& after)
 {
         SString out;
         if (in.len() > maxlen)
-                out = in.substr(0, maxlen / 2) + "..." + in.substr(in.len() - maxlen + maxlen / 2) + SString::sprintf(" (length %d)", in.len());
+                out = in.substr(0, maxlen / 2) + "..." + in.substr(in.len() - maxlen + maxlen / 2);
         else
-                out = in;
+        {
+                out = in; show_length = false;
+        }
         sstringQuote(out);
+        out = before + out + after;
+        if (show_length)
+                out += SString::sprintf(" (length %d)", in.len());
         return out;
 }

cpp/frams/util/sstringutils.h

@@ -19 +19 @@
 bool strContainsOneOf(const char* str, const char* chars);
 bool sstringQuote(SString& target);
-SString sstringShorten(const SString &in, int maxlen);
+SString sstringDelimitAndShorten(const SString &in, int maxlen, bool show_length, const SString& before, const SString& after);
 const char* skipQuoteString(const char* txt, const char* limit);
 int sstringUnquote(SString &target);