changed global variable records to new variant type

apps/esmtool/record.cpp

@@ -713,9 +713,7 @@ void Record<ESM::Faction>::print()
 template<>
 void Record<ESM::Global>::print()
 {
-    // nothing to print (well, nothing that's correct anyway)
+    std::cout << "  " << mData.mValue << std::endl;
-    std::cout << "  Type: " << mData.mType << std::endl;
-    std::cout << "  Value: " << mData.mValue << std::endl;
 }
 
 template<>

apps/opencs/model/doc/document.cpp

@@ -154,8 +154,7 @@ void CSMDoc::Document::addOptionalGlobals()
     {
         ESM::Global global;
         global.mId = sGlobals[i];
-        global.mType = ESM::VT_Int;
+        global.mValue.setType (ESM::VT_Int);
-        global.mValue = 0;
         addOptionalGlobal (global);
     }
 }
@@ -192,9 +191,14 @@ void CSMDoc::Document::createBase()
     for (int i=0; sGlobals[i]; ++i)
     {
         ESM::Global record;
+
         record.mId = sGlobals[i];
-        record.mValue = i==0 ? 1 : 0;
+
-        record.mType = ESM::VT_Float;
+        record.mValue.setType (i==2 ? ESM::VT_Float : ESM::VT_Int);
+
+        if (i==0)
+            record.mValue.setInteger (1);
+
         getData().getGlobals().add (record);
     }
 }

apps/opencs/model/world/columns.hpp

@@ -12,13 +12,13 @@ namespace CSMWorld
 
         virtual QVariant get (const Record<ESXRecordT>& record) const
         {
-            return record.get().mValue;
+            return record.get().mValue.getFloat();
         }
 
         virtual void set (Record<ESXRecordT>& record, const QVariant& data)
         {
             ESXRecordT record2 = record.get();
-            record2.mValue = data.toFloat();
+            record2.mValue.setFloat (data.toFloat());
             record.setModified (record2);
         }
 

apps/openmw/mwworld/globals.cpp

@@ -47,27 +47,24 @@ namespace MWWorld
             char type = ' ';
             Data value;
 
-            switch (iter->mType)
+            switch (iter->mValue.getType())
             {
                 case ESM::VT_Short:
 
                     type = 's';
-                    value.mShort = *reinterpret_cast<const Interpreter::Type_Float *> (
+                    value.mShort = iter->mValue.getInteger();
-                        &iter->mValue);
                     break;
 
                 case ESM::VT_Long:
 
                     type = 'l';
-                    value.mLong = *reinterpret_cast<const Interpreter::Type_Float *> (
+                    value.mLong = iter->mValue.getInteger();
-                        &iter->mValue);
                     break;
 
                 case ESM::VT_Float:
 
                     type = 'f';
-                    value.mFloat = *reinterpret_cast<const Interpreter::Type_Float *> (
+                    value.mFloat = iter->mValue.getFloat();
-                        &iter->mValue);
                     break;
 
                 default:

components/esm/loadglob.cpp

@@ -6,60 +6,23 @@
 namespace ESM
 {
 
-void Global::load(ESMReader &esm)
+    void Global::load(ESMReader &esm)
-{
-    std::string tmp = esm.getHNString("FNAM");
-    if (tmp == "s")
-        mType = VT_Short;
-    else if (tmp == "l")
-        mType = VT_Long;
-    else if (tmp == "f")
-        mType = VT_Float;
-    else
-        esm.fail("Illegal global variable type " + tmp);
-
-    // Note: Both floats and longs are represented as floats.
-    esm.getHNT(mValue, "FLTV");
-
-    if (mType==VT_Short)
     {
-        if (mValue!=mValue)
+        mValue.read (esm, ESM::Variant::Format_Global);
-            mValue = 0; // nan
-        else
-            mValue = static_cast<short> (mValue);
     }
-}
 
-void Global::save(ESMWriter &esm)
+    void Global::save(ESMWriter &esm)
-{
-    switch(mType)
     {
-    case VT_Short:
+        mValue.write (esm, ESM::Variant::Format_Global);
-        esm.writeHNString("FNAM", "s");
-        break;
-
-    case VT_Long:
-        esm.writeHNString("FNAM", "l");
-        break;
-
-    case VT_Float:
-        esm.writeHNString("FNAM", "f");
-        break;
-
-    default:
-        return;
     }
-    esm.writeHNT("FLTV", mValue);
-}
 
     void Global::blank()
     {
-        mValue = 0;
+        mValue.setType (ESM::VT_None);
-        mType = VT_Float;
     }
 
     bool operator== (const Global& left, const Global& right)
     {
-        return left.mId==right.mId && left.mValue==right.mValue && left.mType==right.mType;
+        return left.mId==right.mId && left.mValue==right.mValue;
     }
 }

components/esm/loadglob.hpp

@@ -3,7 +3,6 @@
 
 #include <string>
 
-#include "defs.hpp"
 #include "variant.hpp"
 
 namespace ESM
@@ -19,8 +18,7 @@ class ESMWriter;
 struct Global
 {
     std::string mId;
-    float mValue;
+    Variant mValue;
-    VarType mType;
 
     void load(ESMReader &esm);
     void save(ESMWriter &esm);