libdeng2|Record: Added an explicit setter for char const*

Evidently some compilers (MSVC at least) interpret a zero-length, c-style string as bool, consequently resulting in a NumberValue rather than a TextValue being written to the Record.
skyjake · Mar 13, 2014 · f2e2e15 · f2e2e15
1 parent 077d026
commit f2e2e15
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/doomsday/libdeng2/include/de/data/record.h b/doomsday/libdeng2/include/de/data/record.h
@@ -298,6 +298,9 @@ class DENG2_PUBLIC Record : public ISerializable, public LogEntry::Arg::Base,
      */
     Variable &set(String const &name, bool value);
 
+    /// @copydoc set()
+    Variable &set(String const &name, char const *value);
+
     /// @copydoc set()
     Variable &set(String const &name, Value::Text const &value);
 

diff --git a/doomsday/libdeng2/src/data/record.cpp b/doomsday/libdeng2/src/data/record.cpp
@@ -435,6 +435,15 @@ Variable &Record::set(String const &name, bool value)
     return addBoolean(name, value);
 }
 
+Variable &Record::set(String const &name, char const *value)
+{
+    if(hasMember(name))
+    {
+        return (*this)[name].set(TextValue(value));
+    }
+    return addText(name, value);
+}
+
 Variable &Record::set(String const &name, Value::Text const &value)
 {
     if(hasMember(name))