summaryrefslogtreecommitdiff
path: root/src/chat-server/guildmanager.h
diff options
context:
space:
mode:
authorErik Schilling <ablu.erikschilling@googlemail.com>2012-02-26 14:59:24 +0100
committerErik Schilling <ablu.erikschilling@googlemail.com>2012-02-26 17:11:53 +0100
commit80512086d4e153e883e1cc972bc3240f3cea0891 (patch)
tree8d8e30ab7958d84dd94a000591c626517ce495b4 /src/chat-server/guildmanager.h
parent3629aebb96959afc56cf04d1f2fc4a9f03e94183 (diff)
downloadmanaserv-80512086d4e153e883e1cc972bc3240f3cea0891.tar.gz
manaserv-80512086d4e153e883e1cc972bc3240f3cea0891.tar.bz2
manaserv-80512086d4e153e883e1cc972bc3240f3cea0891.tar.xz
manaserv-80512086d4e153e883e1cc972bc3240f3cea0891.zip
Renamed some iterators + fixed one TODO (stored guilds in a map)
Reviewed-by: bjorn.
Diffstat (limited to 'src/chat-server/guildmanager.h')
-rw-r--r--src/chat-server/guildmanager.h6
1 files changed, 1 insertions, 5 deletions
diff --git a/src/chat-server/guildmanager.h b/src/chat-server/guildmanager.h
index b236b75c..565504ef 100644
--- a/src/chat-server/guildmanager.h
+++ b/src/chat-server/guildmanager.h
@@ -60,10 +60,6 @@ class GuildManager
/**
* Returns the guild with the given id. O(n)
*
- * @todo <b>b_lindeijer:</b> Since this method is used so often, its
- * efficiency should be improved, probably by storing the guilds
- * in a map<int,Guild*> instead of list<Guild*>.
- *
* @return the guild with the given id, or NULL if it doesn't exist
*/
Guild *findById(short id) const;
@@ -110,7 +106,7 @@ class GuildManager
void setUserRights(Guild *guild, int playerId, int rights);
private:
- std::list<Guild*> mGuilds;
+ std::map<int, Guild*> mGuilds;
std::list<int> mOwners;
};