diff --git a/apps/openmw/engine.cpp b/apps/openmw/engine.cpp index c8759a97bc..f16b99f8eb 100644 --- a/apps/openmw/engine.cpp +++ b/apps/openmw/engine.cpp @@ -242,7 +242,7 @@ bool OMW::Engine::frame(float frametime) mScriptManager->getGlobalScripts().run(); } - mWorld->markCellAsUnchanged(); + mWorld->getWorldScene().markCellAsUnchanged(); } if (!guiActive) diff --git a/apps/openmw/mwbase/world.hpp b/apps/openmw/mwbase/world.hpp index 67e8b55db4..ff6ffbc197 100644 --- a/apps/openmw/mwbase/world.hpp +++ b/apps/openmw/mwbase/world.hpp @@ -133,17 +133,6 @@ namespace MWBase virtual void readRecord(ESM::ESMReader& reader, uint32_t type, const std::map& contentFileMap) = 0; - virtual MWWorld::CellStore* getExterior(int x, int y) = 0; - - virtual MWWorld::CellStore* getInterior(std::string_view name) = 0; - - virtual MWWorld::CellStore* getCell(const ESM::CellId& id) = 0; - - virtual bool isCellActive(MWWorld::CellStore* cell) const = 0; - - virtual void testExteriorCells() = 0; - virtual void testInteriorCells() = 0; - virtual void useDeathCamera() = 0; virtual void setWaterHeight(const float height) = 0; @@ -162,9 +151,6 @@ namespace MWBase virtual MWWorld::LocalScripts& getLocalScripts() = 0; - virtual bool hasCellChanged() const = 0; - ///< Has the set of active cells changed, since the last frame? - virtual bool isCellExterior() const = 0; virtual bool isCellQuasiExterior() const = 0; @@ -279,8 +265,6 @@ namespace MWBase virtual const ESM::Cell* getExterior(std::string_view cellName) const = 0; ///< Return a cell matching the given name or a 0-pointer, if there is no such cell. - virtual void markCellAsUnchanged() = 0; - virtual MWWorld::Ptr getFacedObject() = 0; ///< Return pointer to the object the player is looking at, if it is within activation range diff --git a/apps/openmw/mwgui/mapwindow.cpp b/apps/openmw/mwgui/mapwindow.cpp index fe728b3965..83bf6fcae0 100644 --- a/apps/openmw/mwgui/mapwindow.cpp +++ b/apps/openmw/mwgui/mapwindow.cpp @@ -22,6 +22,7 @@ #include "../mwbase/windowmanager.hpp" #include "../mwbase/world.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellstore.hpp" #include "../mwworld/cellutils.hpp" #include "../mwworld/esmstore.hpp" @@ -586,7 +587,7 @@ namespace MWGui if (!entry.mMapTexture) { if (!mInterior) - requestMapRender(MWBase::Environment::get().getWorld()->getExterior(entry.mCellX, entry.mCellY)); + requestMapRender(MWBase::Environment::get().getWorldModel()->getExterior(entry.mCellX, entry.mCellY)); osg::ref_ptr texture = mLocalMapRender->getMapTexture(entry.mCellX, entry.mCellY); if (texture) @@ -624,6 +625,7 @@ namespace MWGui { std::vector doors; MWBase::World* world = MWBase::Environment::get().getWorld(); + MWWorld::Cells* worldModel = MWBase::Environment::get().getWorldModel(); mDoorMarkersToRecycle.insert( mDoorMarkersToRecycle.end(), mInteriorDoorMarkerWidgets.begin(), mInteriorDoorMarkerWidgets.end()); @@ -634,7 +636,7 @@ namespace MWGui for (MyGUI::Widget* widget : mExteriorDoorMarkerWidgets) widget->setVisible(false); - MWWorld::CellStore* cell = world->getInterior(mPrefix); + MWWorld::CellStore* cell = worldModel->getInterior(mPrefix); world->getDoorMarkers(cell, doors); } else @@ -642,7 +644,7 @@ namespace MWGui for (MapEntry& entry : mMaps) { if (!entry.mMapTexture && !widgetCropped(entry.mMapWidget, mLocalMap)) - world->getDoorMarkers(world->getExterior(entry.mCellX, entry.mCellY), doors); + world->getDoorMarkers(worldModel->getExterior(entry.mCellX, entry.mCellY), doors); } if (doors.empty()) return; diff --git a/apps/openmw/mwgui/travelwindow.cpp b/apps/openmw/mwgui/travelwindow.cpp index 5fea9b019b..0df14b7801 100644 --- a/apps/openmw/mwgui/travelwindow.cpp +++ b/apps/openmw/mwgui/travelwindow.cpp @@ -13,6 +13,7 @@ #include "../mwbase/world.hpp" #include "../mwworld/actionteleport.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellstore.hpp" #include "../mwworld/cellutils.hpp" #include "../mwworld/class.hpp" @@ -127,7 +128,7 @@ namespace MWGui if (cellname.empty()) { MWWorld::CellStore* cell - = MWBase::Environment::get().getWorld()->getExterior(cellIndex.x(), cellIndex.y()); + = MWBase::Environment::get().getWorldModel()->getExterior(cellIndex.x(), cellIndex.y()); cellname = MWBase::Environment::get().getWorld()->getCellName(cell); interior = false; } diff --git a/apps/openmw/mwlua/luamanagerimp.cpp b/apps/openmw/mwlua/luamanagerimp.cpp index a6e55ed3e9..d17d420eee 100644 --- a/apps/openmw/mwlua/luamanagerimp.cpp +++ b/apps/openmw/mwlua/luamanagerimp.cpp @@ -27,6 +27,7 @@ #include "../mwworld/class.hpp" #include "../mwworld/esmstore.hpp" #include "../mwworld/ptr.hpp" +#include "../mwworld/scene.hpp" #include "debugbindings.hpp" #include "luabindings.hpp" @@ -402,7 +403,7 @@ namespace MWLua { localScripts = createLocalScripts(ptr); localScripts->addAutoStartedScripts(); - if (ptr.isInCell() && MWBase::Environment::get().getWorld()->isCellActive(ptr.getCell())) + if (ptr.isInCell() && MWBase::Environment::get().getWorldScene()->isCellActive(*ptr.getCell())) mActiveLocalScripts.insert(localScripts); } localScripts->addCustomScript(scriptId); diff --git a/apps/openmw/mwlua/worldview.cpp b/apps/openmw/mwlua/worldview.cpp index 49b28226d2..6435c35415 100644 --- a/apps/openmw/mwlua/worldview.cpp +++ b/apps/openmw/mwlua/worldview.cpp @@ -8,6 +8,7 @@ #include "../mwclass/container.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellutils.hpp" #include "../mwworld/class.hpp" #include "../mwworld/timestamp.hpp" @@ -126,31 +127,30 @@ namespace MWLua // sections. MWWorld::CellStore* WorldView::findCell(const std::string& name, osg::Vec3f position) { - MWBase::World* world = MWBase::Environment::get().getWorld(); - bool exterior = name.empty() || world->getExterior(name); + MWWorld::Cells* cells = MWBase::Environment::get().getWorldModel(); + bool exterior = name.empty() || MWBase::Environment::get().getWorld()->getExterior(name); if (exterior) { const osg::Vec2i cellIndex = MWWorld::positionToCellIndex(position.x(), position.y()); - return world->getExterior(cellIndex.x(), cellIndex.y()); + return cells->getExterior(cellIndex.x(), cellIndex.y()); } else - return world->getInterior(name); + return cells->getInterior(name); } MWWorld::CellStore* WorldView::findNamedCell(const std::string& name) { - MWBase::World* world = MWBase::Environment::get().getWorld(); - const ESM::Cell* esmCell = world->getExterior(name); + MWWorld::Cells* cells = MWBase::Environment::get().getWorldModel(); + const ESM::Cell* esmCell = MWBase::Environment::get().getWorld()->getExterior(name); if (esmCell) - return world->getExterior(esmCell->getGridX(), esmCell->getGridY()); + return cells->getExterior(esmCell->getGridX(), esmCell->getGridY()); else - return world->getInterior(name); + return cells->getInterior(name); } MWWorld::CellStore* WorldView::findExteriorCell(int x, int y) { - MWBase::World* world = MWBase::Environment::get().getWorld(); - return world->getExterior(x, y); + return MWBase::Environment::get().getWorldModel()->getExterior(x, y); } } diff --git a/apps/openmw/mwmechanics/actors.cpp b/apps/openmw/mwmechanics/actors.cpp index b89e32b72c..02f7902454 100644 --- a/apps/openmw/mwmechanics/actors.cpp +++ b/apps/openmw/mwmechanics/actors.cpp @@ -22,6 +22,7 @@ #include "../mwworld/esmstore.hpp" #include "../mwworld/inventorystore.hpp" #include "../mwworld/player.hpp" +#include "../mwworld/scene.hpp" #include "../mwbase/dialoguemanager.hpp" #include "../mwbase/environment.hpp" @@ -1570,7 +1571,8 @@ namespace MWMechanics } else { - const bool cellChanged = world->hasCellChanged(); + MWWorld::Scene* worldScene = MWBase::Environment::get().getWorldScene(); + const bool cellChanged = worldScene->hasCellChanged(); const MWWorld::Ptr actorPtr = actor.getPtr(); // make a copy of the map key to avoid it being // invalidated when the player teleports updateActor(actorPtr, duration); @@ -1582,7 +1584,7 @@ namespace MWMechanics // There needs to be a magic effect update in between. ctrl.updateContinuousVfx(); - if (!cellChanged && world->hasCellChanged()) + if (!cellChanged && worldScene->hasCellChanged()) { return; // for now abort update of the old cell when cell changes by teleportation magic effect // a better solution might be to apply cell changes at the end of the frame diff --git a/apps/openmw/mwscript/cellextensions.cpp b/apps/openmw/mwscript/cellextensions.cpp index 6f5a4c97f0..b5a540fd81 100644 --- a/apps/openmw/mwscript/cellextensions.cpp +++ b/apps/openmw/mwscript/cellextensions.cpp @@ -16,6 +16,7 @@ #include "../mwbase/world.hpp" #include "../mwworld/actionteleport.hpp" #include "../mwworld/cellstore.hpp" +#include "../mwworld/scene.hpp" #include "../mwmechanics/actorutil.hpp" @@ -30,7 +31,7 @@ namespace MWScript public: void execute(Interpreter::Runtime& runtime) override { - runtime.push(MWBase::Environment::get().getWorld()->hasCellChanged() ? 1 : 0); + runtime.push(MWBase::Environment::get().getWorldScene()->hasCellChanged() ? 1 : 0); } }; @@ -50,7 +51,7 @@ namespace MWScript if (wasConsole) MWBase::Environment::get().getWindowManager()->toggleConsole(); - MWBase::Environment::get().getWorld()->testExteriorCells(); + MWBase::Environment::get().getWorldScene()->testExteriorCells(); if (wasConsole) MWBase::Environment::get().getWindowManager()->toggleConsole(); @@ -73,7 +74,7 @@ namespace MWScript if (wasConsole) MWBase::Environment::get().getWindowManager()->toggleConsole(); - MWBase::Environment::get().getWorld()->testInteriorCells(); + MWBase::Environment::get().getWorldScene()->testInteriorCells(); if (wasConsole) MWBase::Environment::get().getWindowManager()->toggleConsole(); diff --git a/apps/openmw/mwscript/transformationextensions.cpp b/apps/openmw/mwscript/transformationextensions.cpp index b2d95c8cea..ee32f9e1ed 100644 --- a/apps/openmw/mwscript/transformationextensions.cpp +++ b/apps/openmw/mwscript/transformationextensions.cpp @@ -13,11 +13,13 @@ #include "../mwbase/environment.hpp" #include "../mwbase/world.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellstore.hpp" #include "../mwworld/cellutils.hpp" #include "../mwworld/class.hpp" #include "../mwworld/manualref.hpp" #include "../mwworld/player.hpp" +#include "../mwworld/scene.hpp" #include "../mwmechanics/actorutil.hpp" @@ -388,6 +390,7 @@ namespace MWScript bool isPlayer = ptr == MWMechanics::getPlayer(); auto world = MWBase::Environment::get().getWorld(); + auto worldModel = MWBase::Environment::get().getWorldModel(); if (isPlayer) { world->getPlayer().setTeleported(true); @@ -396,7 +399,7 @@ namespace MWScript MWWorld::CellStore* store = nullptr; try { - store = world->getInterior(cellID); + store = worldModel->getInterior(cellID); } catch (std::exception&) { @@ -415,7 +418,7 @@ namespace MWScript return; } const osg::Vec2i cellIndex = MWWorld::positionToCellIndex(x, y); - store = world->getExterior(cellIndex.x(), cellIndex.y()); + store = worldModel->getExterior(cellIndex.x(), cellIndex.y()); } if (store) { @@ -432,7 +435,8 @@ namespace MWScript rot.z() = osg::DegreesToRadians(zRot); world->rotateObject(ptr, rot); - ptr.getClass().adjustPosition(ptr, isPlayer || !world->isCellActive(ptr.getCell())); + bool cellActive = MWBase::Environment::get().getWorldScene()->isCellActive(*ptr.getCell()); + ptr.getClass().adjustPosition(ptr, isPlayer || !cellActive); } } }; @@ -470,7 +474,7 @@ namespace MWScript MWWorld::Ptr base = ptr; if (isPlayer) { - MWWorld::CellStore* cell = world->getExterior(cellIndex.x(), cellIndex.y()); + MWWorld::CellStore* cell = MWBase::Environment::get().getWorldModel()->getExterior(cellIndex.x(), cellIndex.y()); ptr = world->moveObject(ptr, cell, osg::Vec3(x, y, z)); } else @@ -487,7 +491,8 @@ namespace MWScript zRot = zRot / 60.0f; rot.z() = osg::DegreesToRadians(zRot); world->rotateObject(ptr, rot); - ptr.getClass().adjustPosition(ptr, isPlayer || !world->isCellActive(ptr.getCell())); + bool cellActive = MWBase::Environment::get().getWorldScene()->isCellActive(*ptr.getCell()); + ptr.getClass().adjustPosition(ptr, isPlayer || !cellActive); } }; @@ -513,13 +518,13 @@ namespace MWScript MWWorld::CellStore* store = nullptr; try { - store = MWBase::Environment::get().getWorld()->getInterior(cellID); + store = MWBase::Environment::get().getWorldModel()->getInterior(cellID); } catch (std::exception&) { const ESM::Cell* cell = MWBase::Environment::get().getWorld()->getExterior(cellID); const osg::Vec2i cellIndex = MWWorld::positionToCellIndex(x, y); - store = MWBase::Environment::get().getWorld()->getExterior(cellIndex.x(), cellIndex.y()); + store = MWBase::Environment::get().getWorldModel()->getExterior(cellIndex.x(), cellIndex.y()); if (!cell) { runtime.getContext().report("unknown cell (" + std::string(cellID) + ")"); @@ -569,7 +574,7 @@ namespace MWScript if (player.getCell()->isExterior()) { const osg::Vec2i cellIndex = MWWorld::positionToCellIndex(x, y); - store = MWBase::Environment::get().getWorld()->getExterior(cellIndex.x(), cellIndex.y()); + store = MWBase::Environment::get().getWorldModel()->getExterior(cellIndex.x(), cellIndex.y()); } else store = player.getCell(); diff --git a/apps/openmw/mwstate/statemanagerimp.cpp b/apps/openmw/mwstate/statemanagerimp.cpp index 68c29252f4..8d13526fb6 100644 --- a/apps/openmw/mwstate/statemanagerimp.cpp +++ b/apps/openmw/mwstate/statemanagerimp.cpp @@ -30,9 +30,11 @@ #include "../mwbase/windowmanager.hpp" #include "../mwbase/world.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellstore.hpp" #include "../mwworld/class.hpp" #include "../mwworld/esmstore.hpp" +#include "../mwworld/scene.hpp" #include "../mwmechanics/actorutil.hpp" #include "../mwmechanics/npcstats.hpp" @@ -554,7 +556,7 @@ void MWState::StateManager::loadGame(const Character* character, const std::file { // Cell no longer exists (i.e. changed game files), choose a default cell Log(Debug::Warning) << "Warning: Player character's cell no longer exists, changing to the default cell"; - MWWorld::CellStore* cell = MWBase::Environment::get().getWorld()->getExterior(0, 0); + MWWorld::CellStore* cell = MWBase::Environment::get().getWorldModel()->getExterior(0, 0); float x, y; MWBase::Environment::get().getWorld()->indexToPosition(0, 0, x, y, false); ESM::Position pos; @@ -575,7 +577,7 @@ void MWState::StateManager::loadGame(const Character* character, const std::file // Since we passed "changeEvent=false" to changeCell, we shouldn't have triggered the cell change flag. // But make sure the flag is cleared anyway in case it was set from an earlier game. - MWBase::Environment::get().getWorld()->markCellAsUnchanged(); + MWBase::Environment::get().getWorldScene()->markCellAsUnchanged(); MWBase::Environment::get().getLuaManager()->gameLoaded(); } diff --git a/apps/openmw/mwworld/actionteleport.cpp b/apps/openmw/mwworld/actionteleport.cpp index 8754e2ba0e..19c6ec65b6 100644 --- a/apps/openmw/mwworld/actionteleport.cpp +++ b/apps/openmw/mwworld/actionteleport.cpp @@ -8,6 +8,7 @@ #include "../mwmechanics/creaturestats.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/cellstore.hpp" #include "../mwworld/cellutils.hpp" #include "../mwworld/class.hpp" @@ -42,6 +43,7 @@ namespace MWWorld void ActionTeleport::teleport(const Ptr& actor) { MWBase::World* world = MWBase::Environment::get().getWorld(); + MWWorld::Cells* worldModel = MWBase::Environment::get().getWorldModel(); actor.getClass().getCreatureStats(actor).land(actor == world->getPlayerPtr()); if (actor == world->getPlayerPtr()) { @@ -58,10 +60,10 @@ namespace MWWorld else if (mCellName.empty()) { const osg::Vec2i index = positionToCellIndex(mPosition.pos[0], mPosition.pos[1]); - world->moveObject(actor, world->getExterior(index.x(), index.y()), mPosition.asVec3(), true, true); + world->moveObject(actor, worldModel->getExterior(index.x(), index.y()), mPosition.asVec3(), true, true); } else - world->moveObject(actor, world->getInterior(mCellName), mPosition.asVec3(), true, true); + world->moveObject(actor, worldModel->getInterior(mCellName), mPosition.asVec3(), true, true); } } diff --git a/apps/openmw/mwworld/player.cpp b/apps/openmw/mwworld/player.cpp index 200f97e543..b01b28a1c2 100644 --- a/apps/openmw/mwworld/player.cpp +++ b/apps/openmw/mwworld/player.cpp @@ -13,6 +13,7 @@ #include #include "../mwworld/esmstore.hpp" +#include "../mwworld/cells.hpp" #include "../mwworld/inventorystore.hpp" #include "../mwworld/magiceffects.hpp" @@ -430,7 +431,7 @@ namespace MWWorld try { - mCellStore = world.getCell(player.mCellId); + mCellStore = MWBase::Environment::get().getWorldModel()->getCell(player.mCellId); } catch (...) { @@ -467,7 +468,7 @@ namespace MWWorld if (player.mHasMark) { mMarkedPosition = player.mMarkedPosition; - mMarkedCell = world.getCell(player.mMarkedCell); + mMarkedCell = MWBase::Environment::get().getWorldModel()->getCell(player.mMarkedCell); } else { diff --git a/apps/openmw/mwworld/scene.cpp b/apps/openmw/mwworld/scene.cpp index 549f77b07a..634e92746d 100644 --- a/apps/openmw/mwworld/scene.cpp +++ b/apps/openmw/mwworld/scene.cpp @@ -543,7 +543,7 @@ namespace MWWorld } mNavigator.setWorldspace( - mWorld.getExterior(playerCellX, playerCellY)->getCell()->mCellId.mWorldspace, navigatorUpdateGuard.get()); + mWorld.getWorldModel().getExterior(playerCellX, playerCellY)->getCell()->mCellId.mWorldspace, navigatorUpdateGuard.get()); mNavigator.updateBounds(pos, navigatorUpdateGuard.get()); mCurrentGridCenter = osg::Vec2i(playerCellX, playerCellY); @@ -566,7 +566,7 @@ namespace MWWorld { if (!isCellInCollection(x, y, collection)) { - refsToLoad += mWorld.getExterior(x, y)->count(); + refsToLoad += mWorld.getWorldModel().getExterior(x, y)->count(); cellsPositionsToLoad.emplace_back(x, y); } } @@ -601,7 +601,7 @@ namespace MWWorld { if (!isCellInCollection(x, y, mActiveCells)) { - CellStore* cell = mWorld.getExterior(x, y); + CellStore* cell = mWorld.getWorldModel().getExterior(x, y); loadCell(cell, loadingListener, changeEvent, pos, navigatorUpdateGuard.get()); } } @@ -610,7 +610,7 @@ namespace MWWorld navigatorUpdateGuard.reset(); - CellStore* current = mWorld.getExterior(playerCellX, playerCellY); + CellStore* current = mWorld.getWorldModel().getExterior(playerCellX, playerCellY); MWBase::Environment::get().getWindowManager()->changeCell(current); if (changeEvent) @@ -663,7 +663,7 @@ namespace MWWorld loadingListener->setLabel( "Testing exterior cells (" + std::to_string(i) + "/" + std::to_string(cells.getExtSize()) + ")..."); - CellStore* cell = mWorld.getExterior(it->mData.mX, it->mData.mY); + CellStore* cell = mWorld.getWorldModel().getExterior(it->mData.mX, it->mData.mY); mNavigator.setWorldspace(cell->getCell()->mCellId.mWorldspace, navigatorUpdateGuard.get()); const osg::Vec3f position = osg::Vec3f(it->mData.mX + 0.5f, it->mData.mY + 0.5f, 0) * Constants::CellSizeInUnits; @@ -720,7 +720,7 @@ namespace MWWorld loadingListener->setLabel( "Testing interior cells (" + std::to_string(i) + "/" + std::to_string(cells.getIntSize()) + ")..."); - CellStore* cell = mWorld.getInterior(it->mName); + CellStore* cell = mWorld.getWorldModel().getInterior(it->mName); mNavigator.setWorldspace(cell->getCell()->mCellId.mWorldspace, navigatorUpdateGuard.get()); ESM::Position position; mWorld.findInteriorPosition(it->mName, position); @@ -842,7 +842,7 @@ namespace MWWorld void Scene::changeToInteriorCell( const std::string& cellName, const ESM::Position& position, bool adjustPlayerPos, bool changeEvent) { - CellStore* cell = mWorld.getInterior(cellName); + CellStore* cell = mWorld.getWorldModel().getInterior(cellName); bool useFading = (mCurrentCell != nullptr); if (useFading) MWBase::Environment::get().getWindowManager()->fadeScreenOut(0.5); @@ -917,7 +917,7 @@ namespace MWWorld changeCellGrid(position.asVec3(), cellIndex.x(), cellIndex.y(), changeEvent); - CellStore* current = mWorld.getExterior(cellIndex.x(), cellIndex.y()); + CellStore* current = mWorld.getWorldModel().getExterior(cellIndex.x(), cellIndex.y()); changePlayerCell(current, position, adjustPlayerPos); if (changeEvent) @@ -1116,12 +1116,12 @@ namespace MWWorld try { if (!door.getCellRef().getDestCell().empty()) - preloadCell(mWorld.getInterior(door.getCellRef().getDestCell())); + preloadCell(mWorld.getWorldModel().getInterior(door.getCellRef().getDestCell())); else { osg::Vec3f pos = door.getCellRef().getDoorDest().asVec3(); const osg::Vec2i cellIndex = positionToCellIndex(pos.x(), pos.y()); - preloadCell(mWorld.getExterior(cellIndex.x(), cellIndex.y()), true); + preloadCell(mWorld.getWorldModel().getExterior(cellIndex.x(), cellIndex.y()), true); exteriorPositions.emplace_back(pos, gridCenterToBounds(getNewGridCenter(pos))); } } @@ -1167,7 +1167,7 @@ namespace MWWorld + mPreloadDistance; if (dist < loadDist) - preloadCell(mWorld.getExterior(cellX + dx, cellY + dy)); + preloadCell(mWorld.getWorldModel().getExterior(cellX + dx, cellY + dy)); } } } @@ -1183,7 +1183,7 @@ namespace MWWorld { for (int dy = -mHalfGridSize; dy <= mHalfGridSize; ++dy) { - mPreloader->preload(mWorld.getExterior(x + dx, y + dy), mRendering.getReferenceTime()); + mPreloader->preload(mWorld.getWorldModel().getExterior(x + dx, y + dy), mRendering.getReferenceTime()); if (++numpreloaded >= mPreloader->getMaxCacheSize()) break; } @@ -1263,12 +1263,12 @@ namespace MWWorld for (ESM::Transport::Dest& dest : listVisitor.mList) { if (!dest.mCellName.empty()) - preloadCell(mWorld.getInterior(dest.mCellName)); + preloadCell(mWorld.getWorldModel().getInterior(dest.mCellName)); else { osg::Vec3f pos = dest.mPos.asVec3(); const osg::Vec2i cellIndex = positionToCellIndex(pos.x(), pos.y()); - preloadCell(mWorld.getExterior(cellIndex.x(), cellIndex.y()), true); + preloadCell(mWorld.getWorldModel().getExterior(cellIndex.x(), cellIndex.y()), true); exteriorPositions.emplace_back(pos, gridCenterToBounds(getNewGridCenter(pos))); } } diff --git a/apps/openmw/mwworld/worldimp.cpp b/apps/openmw/mwworld/worldimp.cpp index 1a547e9387..81a5309add 100644 --- a/apps/openmw/mwworld/worldimp.cpp +++ b/apps/openmw/mwworld/worldimp.cpp @@ -592,39 +592,6 @@ namespace MWWorld return nullptr; } - CellStore* World::getExterior(int x, int y) - { - return mCells.getExterior(x, y); - } - - CellStore* World::getInterior(std::string_view name) - { - return mCells.getInterior(name); - } - - CellStore* World::getCell(const ESM::CellId& id) - { - if (id.mPaged) - return getExterior(id.mIndex.mX, id.mIndex.mY); - else - return getInterior(id.mWorldspace); - } - - bool World::isCellActive(CellStore* cell) const - { - return mWorldScene->getActiveCells().count(cell) > 0; - } - - void World::testExteriorCells() - { - mWorldScene->testExteriorCells(); - } - - void World::testInteriorCells() - { - mWorldScene->testInteriorCells(); - } - void World::useDeathCamera() { mRendering->getCamera()->setMode(MWRender::Camera::Mode::ThirdPerson); @@ -650,11 +617,6 @@ namespace MWWorld return mLocalScripts; } - bool World::hasCellChanged() const - { - return mWorldScene->hasCellChanged(); - } - void World::setGlobalInt(std::string_view name, int value) { bool dateUpdated = mCurrentDate->updateGlobalInt(name, value); @@ -1045,11 +1007,6 @@ namespace MWWorld mCurrentDate->setup(mGlobalVariables); } - void World::markCellAsUnchanged() - { - return mWorldScene->markCellAsUnchanged(); - } - float World::getMaxActivationDistance() const { if (mActivationDistanceOverride >= 0) @@ -1305,7 +1262,7 @@ namespace MWWorld const osg::Vec2i index = positionToCellIndex(position.x(), position.y()); CellStore* cell = ptr.getCell(); - CellStore* newCell = getExterior(index.x(), index.y()); + CellStore* newCell = mCells.getExterior(index.x(), index.y()); bool isCellActive = getPlayerPtr().isInCell() && getPlayerPtr().getCell()->isExterior() && mWorldScene->isCellActive(*newCell); @@ -2850,7 +2807,7 @@ namespace MWWorld pos.rot[0] = pos.rot[1] = pos.rot[2] = 0; pos.pos[0] = pos.pos[1] = pos.pos[2] = 0; - MWWorld::CellStore* cellStore = getInterior(name); + MWWorld::CellStore* cellStore = mCells.getInterior(name); if (!cellStore) return false; @@ -2881,12 +2838,12 @@ namespace MWWorld { ESM::Position doorDest = door->getDoorDest(); const osg::Vec2i index = positionToCellIndex(doorDest.pos[0], doorDest.pos[1]); - source = getExterior(index.x(), index.y()); + source = mCells.getExterior(index.x(), index.y()); } // door to interior else { - source = getInterior(door->getDestCell()); + source = mCells.getInterior(door->getDestCell()); } if (source) { @@ -2933,7 +2890,7 @@ namespace MWWorld if (xResult.ec == std::errc::result_out_of_range || yResult.ec == std::errc::result_out_of_range) throw std::runtime_error("Cell coordinates out of range."); else if (xResult.ec == std::errc{} && yResult.ec == std::errc{}) - ext = getExterior(x, y)->getCell(); + ext = mCells.getExterior(x, y)->getCell(); // ignore std::errc::invalid_argument, as this means that name probably refers to a interior cell // instead of comma separated coordinates } @@ -3352,7 +3309,7 @@ namespace MWWorld nextCells.clear(); for (const std::string& currentCell : currentCells) { - MWWorld::CellStore* next = getInterior(currentCell); + MWWorld::CellStore* next = mCells.getInterior(currentCell); if (!next) continue; @@ -3406,7 +3363,7 @@ namespace MWWorld nextCells.clear(); for (const std::string& cell : currentCells) { - MWWorld::CellStore* next = getInterior(cell); + MWWorld::CellStore* next = mCells.getInterior(cell); checkedCells.insert(cell); if (!next) continue; @@ -3704,7 +3661,7 @@ namespace MWWorld Log(Debug::Warning) << "Failed to confiscate items: prison marker not linked to prison interior"; return; } - MWWorld::CellStore* prison = getInterior(prisonName); + MWWorld::CellStore* prison = mCells.getInterior(prisonName); if (!prison) { Log(Debug::Warning) << "Failed to confiscate items: failed to load cell " << prisonName; diff --git a/apps/openmw/mwworld/worldimp.hpp b/apps/openmw/mwworld/worldimp.hpp index 5c227d195b..f7245b18b1 100644 --- a/apps/openmw/mwworld/worldimp.hpp +++ b/apps/openmw/mwworld/worldimp.hpp @@ -219,17 +219,6 @@ namespace MWWorld void readRecord(ESM::ESMReader& reader, uint32_t type, const std::map& contentFileMap) override; - CellStore* getExterior(int x, int y) override; - - CellStore* getInterior(std::string_view name) override; - - CellStore* getCell(const ESM::CellId& id) override; - - bool isCellActive(CellStore* cell) const override; - - void testExteriorCells() override; - void testInteriorCells() override; - // switch to POV before showing player's death animation void useDeathCamera() override; @@ -253,9 +242,6 @@ namespace MWWorld LocalScripts& getLocalScripts() override; - bool hasCellChanged() const override; - ///< Has the set of active cells changed, since the last frame? - bool isCellExterior() const override; bool isCellQuasiExterior() const override; @@ -375,8 +361,6 @@ namespace MWWorld const ESM::Cell* getExterior(std::string_view cellName) const override; ///< Return a cell matching the given name or a 0-pointer, if there is no such cell. - void markCellAsUnchanged() override; - MWWorld::Ptr getFacedObject() override; ///< Return pointer to the object the player is looking at, if it is within activation range