modified Npc and Creature class to let them store a copy of their own ID

apps/esmtool/esmtool.cpp

@@ -180,7 +180,7 @@ int main(int argc, char**argv)
         case REC_CREA:
           {
             Creature b;
-            b.load(esm);
+            b.load(esm, id);
             if(quiet) break;
             cout << "  Name: " << b.name << endl;
             break;

apps/openmw/mwworld/manualref.hpp

@@ -36,6 +36,23 @@ namespace MWWorld
                 return false;
             }
 
+            template<typename T>
+            bool create (const ESMS::RecListWithIDT<T>& list, const std::string& name)
+            {
+                if (const T *instance = list.search (name))
+                {
+                    ESMS::LiveCellRef<T, RefData> ref;
+                    ref.base = instance;
+
+                    mRef = ref;
+                    mPtr = Ptr (&boost::any_cast<ESMS::LiveCellRef<T, RefData>&> (mRef), 0);
+
+                    return true;
+                }
+
+                return false;
+            }
+
         public:
 
             ManualRef (const ESMS::ESMStore& store, const std::string& name)

components/esm/loadcrea.hpp

@@ -62,8 +62,12 @@ struct Creature
   // Defined in loadcont.hpp
   InventoryList inventory;
 
-  void load(ESMReader &esm)
+    std::string mId;
+
+  void load(ESMReader &esm, const std::string& id)
   {
+    mId = id;
+
     model = esm.getHNString("MODL");
     original = esm.getHNOString("CNAM");
     name = esm.getHNOString("FNAM");

components/esm/loadnpc.hpp

@@ -97,8 +97,12 @@ struct NPC
   std::string name, model, race, cls, faction, script,
     hair, head; // body parts
 
-  void load(ESMReader &esm)
+    std::string mId;
+
+  void load(ESMReader &esm, const std::string& id)
   {
+    mId = id;
+
     npdt52.gold = -10;
 
     model = esm.getHNOString("MODL");

components/esm_store/reclists.hpp

@@ -68,6 +68,44 @@ namespace ESMS
     int getSize() { return list.size(); }
   };
 
+    /// Modified version of RecListT for records, that need to store their own ID
+  template <typename X>
+  struct RecListWithIDT : RecList
+  {
+    typedef std::map<std::string,X> MapType;
+
+    MapType list;
+
+    // Load one object of this type
+    void load(ESMReader &esm, const std::string &id)
+    {
+      std::string id2 = toLower (id);
+      list[id2].load(esm, id2);
+    }
+
+    // Find the given object ID, or return NULL if not found.
+    const X* search(const std::string &id) const
+    {
+      std::string id2 = toLower (id);
+      if(list.find(id2) == list.end())
+        return NULL;
+      return &list.find(id2)->second;
+    }
+
+    // Find the given object ID (throws an exception if not found)
+    const X* find(const std::string &id) const
+    {
+        const X *object = search (id);
+
+        if (!object)
+            throw std::runtime_error ("object " + id + " not found");
+
+        return object;
+    }
+
+    int getSize() { return list.size(); }
+  };
+
   // The only difference to the above is a slight change to the load()
   // function. We might merge these together later, and store the id
   // in all the structs.

components/esm_store/store.hpp

@@ -40,7 +40,7 @@ namespace ESMS
     RecListT<Clothing>          clothes;
     RecListT<LoadCNTC>          contChange;
     RecListT<Container>         containers;
-    RecListT<Creature>          creatures;
+    RecListWithIDT<Creature>          creatures;
     RecListT<LoadCREC>          creaChange;
     RecListT<Dialogue>          dialogs;
     RecListT<Door>              doors;
@@ -53,7 +53,7 @@ namespace ESMS
     RecListT<Light>             lights;
     RecListT<Tool>              lockpicks;
     RecListT<Misc>              miscItems;
-    RecListT<NPC>               npcs;
+    RecListWithIDT<NPC>               npcs;
     RecListT<LoadNPCC>          npcChange;
     RecListT<Probe>             probes;
     RecListT<Race>              races;