Changeset 950 for cpp/frams/param/param.cpp

Timestamp:

06/20/20 01:02:20 (4 years ago)

Author:

Maciej Komosinski

Message:

Improved error messages (display longer names of the relevant object and field when available)

File:

• cpp/frams/param/param.cpp (modified) (10 diffs)

cpp/frams/param/param.cpp

@@ -331 +331 @@
 }
 
+SString ParamInterface::nameDotProperty(int prop)
+{
+        SString ret = getName();
+        ret+=".";
+        ret+=id(prop);
+        return ret;
+}
+
+SString ParamInterface::nameDotPropertyForMessages(int prop)
+{
+        SString name_dot_prop = nameDotProperty(prop);
+        if (strcmp(getName(),getLongName())==0)
+                {
+                if (strcmp(id(prop),name(prop))==0)
+                        return name_dot_prop;
+                else
+                        return SString("'")+name(prop)+"': "+name_dot_prop;
+                }
+        else
+                return SString("'")+name(prop)+"' in '"+getLongName()+"': "+name_dot_prop;
+}
 
 int SimpleAbstractParam::isequal(int i, void* defdata)
@@ -421 +442 @@
         SString field;
         if (field_index >= 0)
-                field = SString::sprintf("'%s.%s'", pi->getName(), pi->id(field_index));
+                field = pi->nameDotPropertyForMessages(field_index);
         else
                 field = SString::sprintf("unknown property of '%s'", pi->getName());
@@ -445 +466 @@
                 bool show_length = valuetoset.getType() == TString;
                 const char* quote = (valuetoset.getType() == TString) ? "\"" : "'";
-                logPrintf("Param", "set", LOG_WARN, "Setting %s.%s = %s exceeded allowed range (too %s). %s to %s.",
-                        getName(), id(i),
+                logPrintf("Param", "set", LOG_WARN, "Setting %s = %s exceeded allowed range (too %s). %s to %s.",
+                        nameDotPropertyForMessages(i).c_str(),
                         ::sstringDelimitAndShorten(svaluetoset, 30, show_length, quote, quote).c_str(),
                         (setflags&PSET_HITMAX) ? (s_type ? "long" : "big") : "small", s_type ? "Truncated" : "Adjusted",
@@ -674 +695 @@
         case 'o':       ret.setObject(getObject(i)); break;
         case 'x':       ret = getExtValue(i); break;
-        default: logPrintf("ParamInterface", "get", LOG_ERROR, "'%s.%s' is not a property", getName(), id(i));
+        default: logPrintf("ParamInterface", "get", LOG_ERROR, "%s is not a property", nameDotPropertyForMessages(i).c_str());
         }
 }
@@ -718 +739 @@
                         if (v.type == TObj)
                         {
-                                logPrintf("ParamInterface", "set", LOG_ERROR, "Setting int '%s.%s' from object reference (%s)", getName(), id(i), v.getString().c_str());
+                                logPrintf("ParamInterface", "set", LOG_ERROR, "Setting int %s from object reference (%s)", nameDotPropertyForMessages(i).c_str(), v.getString().c_str());
                                 return 0;
                         }
@@ -730 +751 @@
                         if (v.type == TObj)
                         {
-                                logPrintf("ParamInterface", "set", LOG_ERROR, "Setting float '%s.%s' from object reference (%s)", getName(), id(i), v.getString().c_str());
+                                logPrintf("ParamInterface", "set", LOG_ERROR, "Setting float %s from object reference (%s)", nameDotPropertyForMessages(i).c_str(), v.getString().c_str());
                                 return 0;
                         }
@@ -739 +760 @@
         case 'o':
                 if ((v.type != TUnknown) && (v.type != TObj))
-                        logPrintf("ParamInterface", "set", LOG_ERROR, "Setting object '%s.%s' from %s", getName(), id(i), v.typeAndValue().c_str());
+                        logPrintf("ParamInterface", "set", LOG_ERROR, "Setting object %s from %s", nameDotPropertyForMessages(i).c_str(), v.typeAndValue().c_str());
                 else
                         return setObject(i, v.getObject());
                 break;
         case 'x': return setExtValue(i, v);
-        default: logPrintf("ParamInterface", "set", LOG_ERROR, "'%s.%s' is not a property", getName(), id(i));
+        default: logPrintf("ParamInterface", "set", LOG_ERROR, "%s is not a property", nameDotPropertyForMessages(i).c_str());
         }
         return 0;
@@ -922 +943 @@
         if (err != NULL)
                 logPrintf("SimpleAbstractParam", "sanityCheck", LOG_ERROR,
-                        "Invalid ParamEntry for %s.%s (%s)", getName(), pe->id, err);
+                          "Invalid ParamEntry for %s (%s)", nameDotPropertyForMessages(i).c_str(), err);
 }
 #endif
@@ -1156 +1177 @@
                 else
                         logPrintf("SimpleAbstractParam", "setObject", LOG_ERROR,
-                                "'%s.%s' is PARAM_OBJECTSET but no 'assign()' in %s", getName(), pe->id, o.interfaceName());
+                                "%s is PARAM_OBJECTSET but no 'assign()' in %s", nameDotPropertyForMessages(i).c_str(), o.interfaceName());
                 return PSET_CHANGED;
         }
@@ -1205 +1226 @@
         {
                 logPrintf("SimpleAbstractParam", "call", LOG_ERROR,
-                        (*pe->type != 'p') ? "'%s.%s' is not a function" : "Internal error - undefined function pointer for '%s.%s'", getName(), pe->id);
+                        (*pe->type != 'p') ? "%s is not a function" : "Internal error - undefined function pointer for %s", nameDotPropertyForMessages(i).c_str());
                 ret->setInvalid();
         }