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

Fix for "GlobalMap trying to erase an inactive camera" spam in main menu

Browse source
This commit is contained in:
scrawl 2015-06-04 16:32:21 +02:00
parent e046119487
commit ce0d93caed
2 changed files with 28 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
apps/openmw/mwrender/globalmap.cpp
View file

@ -63,17 +63,31 @@ namespace
{
public:
CameraUpdateCallback(osg::Camera* cam, MWRender::GlobalMap* parent)
: mCamera(cam), mParent(parent)
: mRendered(false)
, mCamera(cam)
, mParent(parent)
{
}
virtual void operator()(osg::Node* node, osg::NodeVisitor* nv)
{
mParent->markForRemoval(mCamera);
if (mRendered)
{
mCamera->setNodeMask(0);
return;
}
traverse(node, nv);
if (!mRendered)
{
mRendered = true;
mParent->markForRemoval(mCamera);
}
}
private:
bool mRendered;
osg::ref_ptr<osg::Camera> mCamera;
MWRender::GlobalMap* mParent;
};

14
apps/openmw/mwrender/localmap.cpp
View file

@ -30,13 +30,22 @@ namespace
{
public:
CameraUpdateCallback(osg::Camera* cam, MWRender::LocalMap* parent)
: mCamera(cam), mParent(parent)
: mRendered(false)
, mCamera(cam)
, mParent(parent)
{
}
virtual void operator()(osg::Node*, osg::NodeVisitor*)
{
mParent->markForRemoval(mCamera);
if (mRendered)
mCamera->setNodeMask(0);
if (!mRendered)
{
mRendered = true;
mParent->markForRemoval(mCamera);
}
// Note, we intentionally do not traverse children here. The map camera's scene data is the same as the master camera's,
// so it has been updated already.
@ -44,6 +53,7 @@ namespace
}
private:
bool mRendered;
osg::ref_ptr<osg::Camera> mCamera;
MWRender::LocalMap* mParent;
};