diff options
author | Erik Schilling <ablu.erikschilling@googlemail.com> | 2012-02-26 14:59:24 +0100 |
---|---|---|
committer | Erik Schilling <ablu.erikschilling@googlemail.com> | 2012-02-26 17:11:53 +0100 |
commit | 80512086d4e153e883e1cc972bc3240f3cea0891 (patch) | |
tree | 8d8e30ab7958d84dd94a000591c626517ce495b4 /src/chat-server/guildmanager.h | |
parent | 3629aebb96959afc56cf04d1f2fc4a9f03e94183 (diff) | |
download | manaserv-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.h | 6 |
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; }; |