Archive/openmw
0
0
Fork 0
mirror of https://gitlab.com/OpenMW/openmw.git synced 2025-04-28 21:07:59 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions 40

Merge deleted refs when unloading a cell

Browse source 
To unload objects scheduled to be teleported.
This commit is contained in:
elsid 2025-03-08 12:47:07 +01:00
parent 5f92d520ee
commit f800f63ee5
No known key found for this signature in database
GPG key ID: B845CB9FEE18AB40
2 changed files with 13 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

18
apps/openmw/mwworld/cellstore.cpp
View file

@ -337,13 +337,13 @@ namespace
// helper function for forEachInternal // helper function for forEachInternal
template <class Visitor, class List> template <class Visitor, class List>
bool forEachImp(Visitor& visitor, List& list, MWWorld::CellStore* cellStore) bool forEachImp(Visitor& visitor, List& list, MWWorld::CellStore& cellStore, bool includeDeleted)
{ {
for (typename List::List::iterator iter(list.mList.begin()); iter != list.mList.end(); ++iter) for (auto& v : list.mList)
{ {
if (!MWWorld::CellStore::isAccessible(iter->mData, iter->mRef)) if (!includeDeleted && !MWWorld::CellStore::isAccessible(v.mData, v.mRef))
continue; continue;
if (!visitor(MWWorld::Ptr(&*iter, cellStore))) if (!visitor(MWWorld::Ptr(&v, &cellStore)))
return false; return false;
} }
return true; return true;
@ -399,12 +399,12 @@ namespace MWWorld
// listing only objects owned by this cell. Internal use only, you probably want to use forEach() so that moved // listing only objects owned by this cell. Internal use only, you probably want to use forEach() so that moved
// objects are accounted for. // objects are accounted for.
template <class Visitor> template <class Visitor>
static bool forEachInternal(Visitor& visitor, MWWorld::CellStore& cellStore) static bool forEachInternal(Visitor& visitor, MWWorld::CellStore& cellStore, bool includeDeleted)
{ {
bool returnValue = true; bool returnValue = true;
Misc::tupleForEach(cellStore.mCellStoreImp->mRefLists, [&visitor, &returnValue, &cellStore](auto& store) { Misc::tupleForEach(cellStore.mCellStoreImp->mRefLists, [&](auto& store) {
returnValue = returnValue && forEachImp(visitor, store, &cellStore); returnValue = returnValue && forEachImp(visitor, store, cellStore, includeDeleted);
}); });
return returnValue; return returnValue;
@ -583,11 +583,11 @@ namespace MWWorld
mMergedRefsNeedsUpdate = true; mMergedRefsNeedsUpdate = true;
} }
void CellStore::updateMergedRefs() const void CellStore::updateMergedRefs(bool includeDeleted) const
{ {
mMergedRefs.clear(); mMergedRefs.clear();
MergeVisitor visitor(mMergedRefs, mMovedHere, mMovedToAnotherCell); MergeVisitor visitor(mMergedRefs, mMovedHere, mMovedToAnotherCell);
CellStoreImp::forEachInternal(visitor, const_cast<CellStore&>(*this)); CellStoreImp::forEachInternal(visitor, const_cast<CellStore&>(*this), includeDeleted);
visitor.merge(); visitor.merge();
mMergedRefsNeedsUpdate = false; mMergedRefsNeedsUpdate = false;
} }

8
apps/openmw/mwworld/cellstore.hpp
View file

@ -219,7 +219,7 @@ namespace MWWorld
return false; return false;
if (mMergedRefsNeedsUpdate) if (mMergedRefsNeedsUpdate)
updateMergedRefs(); updateMergedRefs(includeDeleted);
if (mMergedRefs.empty()) if (mMergedRefs.empty())
return true; return true;
@ -248,7 +248,7 @@ namespace MWWorld
return false; return false;
if (mMergedRefsNeedsUpdate) if (mMergedRefsNeedsUpdate)
updateMergedRefs(); updateMergedRefs(includeDeleted);
for (const LiveCellRefBase* mergedRef : mMergedRefs) for (const LiveCellRefBase* mergedRef : mMergedRefs)
{ {
@ -273,7 +273,7 @@ namespace MWWorld
return false; return false;
if (mMergedRefsNeedsUpdate) if (mMergedRefsNeedsUpdate)
updateMergedRefs(); updateMergedRefs(includeDeleted);
if (mMergedRefs.empty()) if (mMergedRefs.empty())
return true; return true;
@ -403,7 +403,7 @@ namespace MWWorld
/// Repopulate mMergedRefs. /// Repopulate mMergedRefs.
void requestMergedRefsUpdate(); void requestMergedRefsUpdate();
void updateMergedRefs() const; void updateMergedRefs(bool includeDeleted = false) const;
// (item, max charge) // (item, max charge)
typedef std::vector<std::pair<LiveCellRefBase*, float>> TRechargingItems; typedef std::vector<std::pair<LiveCellRefBase*, float>> TRechargingItems;