krageon/openmw-tes3coop
1
0
Fork 0
forked from teamnwah/openmw-tes3coop
Code Issues Pull requests Releases Wiki Activity

Avoid redundant allocations in Store::search

Browse source
This commit is contained in:
scrawl 2017-02-23 17:18:27 +01:00
parent 6e996bf2a3
commit 25ca89b560
1 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
apps/openmw/mwworld/store.cpp
View file

@ -136,15 +136,14 @@ namespace MWWorld
template<typename T>
const T *Store<T>::search(const std::string &id) const
{
T item;
item.mId = Misc::StringUtils::lowerCase(id);
std::string idLower = Misc::StringUtils::lowerCase(id);
typename Dynamic::const_iterator dit = mDynamic.find(item.mId);
typename Dynamic::const_iterator dit = mDynamic.find(idLower);
if (dit != mDynamic.end()) {
return &dit->second;
}
typename std::map<std::string, T>::const_iterator it = mStatic.find(item.mId);
typename std::map<std::string, T>::const_iterator it = mStatic.find(idLower);
if (it != mStatic.end() && Misc::StringUtils::ciEqual(it->second.mId, id)) {
return &(it->second);
@ -274,10 +273,9 @@ namespace MWWorld
template<typename T>
bool Store<T>::eraseStatic(const std::string &id)
{
T item;
item.mId = Misc::StringUtils::lowerCase(id);
std::string idLower = Misc::StringUtils::lowerCase(id);
typename std::map<std::string, T>::iterator it = mStatic.find(item.mId);
typename std::map<std::string, T>::iterator it = mStatic.find(idLower);
if (it != mStatic.end() && Misc::StringUtils::ciEqual(it->second.mId, id)) {
// delete from the static part of mShared
@ -285,7 +283,7 @@ namespace MWWorld
typename std::vector<T *>::iterator end = sharedIter + mStatic.size();
while (sharedIter != mShared.end() && sharedIter != end) {
if((*sharedIter)->mId == item.mId) {
if((*sharedIter)->mId == idLower) {
mShared.erase(sharedIter);
break;
}