From 0485bd935b235e1162332ca145e7fbb6e5aecb3b Mon Sep 17 00:00:00 2001
From: deva <deva>
Date: Thu, 14 Jan 2010 14:11:27 +0000
Subject: Made EntityList able to handle several files with same macroname and
 version number, without corrupting the internal data structure (switched from
 map to multimap).

---
 server/src/entitylist.cc   | 4 ++--
 server/src/entitylist.h    | 3 ++-
 server/src/macrolist.cc    | 5 ++++-
 server/src/templatelist.cc | 3 ++-
 4 files changed, 10 insertions(+), 5 deletions(-)

(limited to 'server/src')

diff --git a/server/src/entitylist.cc b/server/src/entitylist.cc
index 64bd49d..b84df61 100644
--- a/server/src/entitylist.cc
+++ b/server/src/entitylist.cc
@@ -139,8 +139,8 @@ bool EntityList::removeFile(std::string file)
     while(j != i->second.end()) {
       if(file == j->second) {
         PRACRO_DEBUG(entitylist, "Removing file: %s\n", file.c_str());
-        i->second.erase(j->first);
-        if(i->second.size() == 0) erase(i->first);
+        i->second.erase(j);
+        if(i->second.size() == 0) erase(i);
         return true;
       }
       j++;
diff --git a/server/src/entitylist.h b/server/src/entitylist.h
index b596c07..9185f67 100644
--- a/server/src/entitylist.h
+++ b/server/src/entitylist.h
@@ -30,6 +30,7 @@
 
 #include <map>
 #include <string>
+
 #include "versionstr.h"
 #include "mutex.h"
 
@@ -40,7 +41,7 @@
 /**
  * The Items contained in the EntityList.
  */
-typedef std::map<VersionStr, std::string> EntityListItem;
+typedef std::multimap<VersionStr, std::string> EntityListItem;
 
 /**
  * The EntityList class is intended for entity file caching, so that all entitys
diff --git a/server/src/macrolist.cc b/server/src/macrolist.cc
index 86584d4..9dadb12 100644
--- a/server/src/macrolist.cc
+++ b/server/src/macrolist.cc
@@ -27,6 +27,8 @@
  */
 #include "macrolist.h"
 
+#include <utility>
+
 #include "macroheaderparser.h"
 
 #include "debug.h"
@@ -45,7 +47,8 @@ void MacroList::addFile(std::string file)
   try {
     parser.parse();
     Macro *macro = parser.getMacro();
-    (*this)[macro->attributes["name"]][VersionStr(macro->attributes["version"])] = file;
+    std::pair<VersionStr, std::string> p(VersionStr(macro->attributes["version"]), file);
+    (*this)[macro->attributes["name"]].insert(p);
   } catch(Exception &e) {
     PRACRO_WARN(macrolist, "Skipping %s: %s\n", file.c_str(), e.what());
   }
diff --git a/server/src/templatelist.cc b/server/src/templatelist.cc
index b952467..0fc8ec5 100644
--- a/server/src/templatelist.cc
+++ b/server/src/templatelist.cc
@@ -44,7 +44,8 @@ void TemplateList::addFile(std::string file)
   try {
     parser.parse();
     Template *templ = parser.getTemplate();
-    (*this)[templ->attributes["name"]][VersionStr(templ->attributes["version"])] = file;
+    std::pair<VersionStr, std::string> p(VersionStr(templ->attributes["version"]), file);
+    (*this)[templ->attributes["name"]].insert(p);
   } catch(Exception &e) {
     PRACRO_WARN(templatelist, "Skipping %s: %s\n", file.c_str(), e.what());
   }
-- 
cgit v1.2.3