From a7102c143f2f44701c95e2824ab93a459bb5d162 Mon Sep 17 00:00:00 2001
From: Marc Zinnschlag <marc@zpages.de>
Date: Tue, 26 Feb 2013 14:33:05 +0100
Subject: [PATCH] ESX variable type cleanup

---
 apps/openmw/mwworld/globals.cpp | 66 ++++++++++++++++-----------------
 components/esm/defs.hpp         |  4 +-
 components/esm/loadglob.cpp     |  4 +-
 3 files changed, 37 insertions(+), 37 deletions(-)

diff --git a/apps/openmw/mwworld/globals.cpp b/apps/openmw/mwworld/globals.cpp
index f010661b9..8742dd892 100644
--- a/apps/openmw/mwworld/globals.cpp
+++ b/apps/openmw/mwworld/globals.cpp
@@ -21,21 +21,21 @@ namespace MWWorld
     Globals::Collection::const_iterator Globals::find (const std::string& name) const
     {
         Collection::const_iterator iter = mVariables.find (name);
-            
+
         if (iter==mVariables.end())
             throw std::runtime_error ("unknown global variable: " + name);
-            
-        return iter;    
+
+        return iter;
     }
 
     Globals::Collection::iterator Globals::find (const std::string& name)
     {
         Collection::iterator iter = mVariables.find (name);
-            
+
         if (iter==mVariables.end())
             throw std::runtime_error ("unknown global variable: " + name);
-            
-        return iter;    
+
+        return iter;
     }
 
     Globals::Globals (const MWWorld::ESMStore& store)
@@ -46,44 +46,44 @@ namespace MWWorld
         {
             char type = ' ';
             Data value;
-        
+
             switch (iter->mType)
             {
                 case ESM::VT_Short:
-                    
+
                     type = 's';
                     value.mShort = *reinterpret_cast<const Interpreter::Type_Float *> (
                         &iter->mValue);
                     break;
-                    
-                case ESM::VT_Int:
-                
+
+                case ESM::VT_Long:
+
                     type = 'l';
                     value.mLong = *reinterpret_cast<const Interpreter::Type_Float *> (
                         &iter->mValue);
                     break;
-                    
+
                 case ESM::VT_Float:
-                
+
                     type = 'f';
                     value.mFloat = *reinterpret_cast<const Interpreter::Type_Float *> (
                         &iter->mValue);
                     break;
-                    
+
                 default:
-                
+
                     throw std::runtime_error ("unsupported global variable type");
-            }            
+            }
 
             mVariables.insert (std::make_pair (iter->mId, std::make_pair (type, value)));
         }
-        
+
         if (mVariables.find ("dayspassed")==mVariables.end())
         {
             // vanilla Morrowind does not define dayspassed.
             Data value;
             value.mLong = 0;
-            
+
             mVariables.insert (std::make_pair ("dayspassed", std::make_pair ('l', value)));
         }
     }
@@ -91,31 +91,31 @@ namespace MWWorld
     const Globals::Data& Globals::operator[] (const std::string& name) const
     {
         Collection::const_iterator iter = find (name);
-            
+
         return iter->second.second;
     }
 
     Globals::Data& Globals::operator[] (const std::string& name)
     {
         Collection::iterator iter = find (name);
-            
+
         return iter->second.second;
     }
-    
+
     void Globals::setInt (const std::string& name, int value)
     {
         Collection::iterator iter = find (name);
-        
+
         switch (iter->second.first)
         {
             case 's': iter->second.second.mShort = value; break;
             case 'l': iter->second.second.mLong = value; break;
             case 'f': iter->second.second.mFloat = value; break;
-            
+
             default: throw std::runtime_error ("unsupported global variable type");
         }
     }
-    
+
     void Globals::setFloat (const std::string& name, float value)
     {
         Collection::iterator iter = find (name);
@@ -127,9 +127,9 @@ namespace MWWorld
             case 'f': iter->second.second.mFloat = value; break;
 
             default: throw std::runtime_error ("unsupported global variable type");
-        }        
+        }
     }
-    
+
     int Globals::getInt (const std::string& name) const
     {
         Collection::const_iterator iter = find (name);
@@ -141,13 +141,13 @@ namespace MWWorld
             case 'f': return iter->second.second.mFloat;
 
             default: throw std::runtime_error ("unsupported global variable type");
-        }                
+        }
     }
-        
+
     float Globals::getFloat (const std::string& name) const
     {
         Collection::const_iterator iter = find (name);
-    
+
         switch (iter->second.first)
         {
             case 's': return iter->second.second.mShort;
@@ -155,16 +155,16 @@ namespace MWWorld
             case 'f': return iter->second.second.mFloat;
 
             default: throw std::runtime_error ("unsupported global variable type");
-        }    
+        }
     }
-    
+
     char Globals::getType (const std::string& name) const
     {
         Collection::const_iterator iter = mVariables.find (name);
-        
+
         if (iter==mVariables.end())
             return ' ';
-            
+
         return iter->second.first;
     }
 }
diff --git a/components/esm/defs.hpp b/components/esm/defs.hpp
index aa870f925..143d90034 100644
--- a/components/esm/defs.hpp
+++ b/components/esm/defs.hpp
@@ -13,9 +13,9 @@ enum VarType
 {
     VT_Unknown,
     VT_None,
-    VT_Short,
+    VT_Short, // stored as a float, kinda
     VT_Int,
-    VT_Long,
+    VT_Long, // stored as a float
     VT_Float,
     VT_String,
     VT_Ignored
diff --git a/components/esm/loadglob.cpp b/components/esm/loadglob.cpp
index ceaa86948..429c6ff1d 100644
--- a/components/esm/loadglob.cpp
+++ b/components/esm/loadglob.cpp
@@ -12,7 +12,7 @@ void Global::load(ESMReader &esm)
     if (tmp == "s")
         mType = VT_Short;
     else if (tmp == "l")
-        mType = VT_Int;
+        mType = VT_Long;
     else if (tmp == "f")
         mType = VT_Float;
     else
@@ -30,7 +30,7 @@ void Global::save(ESMWriter &esm)
         esm.writeHNString("FNAM", "s");
         break;
 
-    case VT_Int:
+    case VT_Long:
         esm.writeHNString("FNAM", "l");
         break;