60078c34c7
This is a classic example of why naked pointers suck! The right way to fix an interdependency would be to have these objects hold onto weak references to each other. But that's way too big a change at this point in time, so I've added this ugly hack instead. This will fix an intermittent crash when either accessing the library or exiting a match.
96 lines
2.1 KiB
C++
96 lines
2.1 KiB
C++
#include "PrecompiledHeader.h"
|
|
|
|
#include "GuiLayers.h"
|
|
#include "Player.h"
|
|
|
|
GuiLayer::GuiLayer()
|
|
{
|
|
modal = 0;
|
|
hasFocus = false;
|
|
mCount = 0;
|
|
mCurr = 0;
|
|
mActionButton = JGE_BTN_OK;
|
|
}
|
|
|
|
GuiLayer::~GuiLayer()
|
|
{
|
|
resetObjects();
|
|
}
|
|
|
|
void GuiLayer::Add(JGuiObject *object)
|
|
{
|
|
mObjects.push_back(object);
|
|
mCount++;
|
|
}
|
|
|
|
int GuiLayer::Remove(JGuiObject *object)
|
|
{
|
|
for (int i = 0; i < mCount; i++)
|
|
if (mObjects[i] == object)
|
|
{
|
|
delete mObjects[i];
|
|
mObjects.erase(mObjects.begin() + i);
|
|
mCount--;
|
|
if (mCurr == mCount)
|
|
mCurr = 0;
|
|
return 1;
|
|
}
|
|
return 0;
|
|
}
|
|
|
|
int GuiLayer::getMaxId()
|
|
{
|
|
return mCount;
|
|
}
|
|
|
|
void GuiLayer::Render()
|
|
{
|
|
for (int i = 0; i < mCount; i++)
|
|
if (mObjects[i] != NULL)
|
|
mObjects[i]->Render();
|
|
}
|
|
|
|
void GuiLayer::Update(float dt)
|
|
{
|
|
for (int i = 0; i < mCount; i++)
|
|
if (mObjects[i] != NULL)
|
|
mObjects[i]->Update(dt);
|
|
}
|
|
|
|
void GuiLayer::resetObjects()
|
|
{
|
|
for (int i = 0; i < mCount; i++)
|
|
if (mObjects[i])
|
|
{
|
|
// big, ugly hack around CardView / MTGCardInstance ownership problem - these two classes have an interdependency with naked pointers,
|
|
// but the order of destruction can leave a dangling pointer reference inside of a CardView, which attempts to clean up its own pointer
|
|
// reference in an MTGCardInstance when it's destroyed. Ideally, CardView should only hold onto a weak reference, but that's a bigger overhaul.
|
|
// For now, if we get here, clear out the MTGCardInstance pointer of a CardView before calling delete.
|
|
CardView* cardView = dynamic_cast<CardView*>(mObjects[i]);
|
|
if (cardView)
|
|
{
|
|
cardView->card = NULL;
|
|
}
|
|
delete mObjects[i];
|
|
}
|
|
mObjects.clear();
|
|
mCount = 0;
|
|
mCurr = 0;
|
|
}
|
|
|
|
int GuiLayer::getIndexOf(JGuiObject * object)
|
|
{
|
|
for (int i = 0; i < mCount; i++)
|
|
{
|
|
if (mObjects[i] == object)
|
|
return i;
|
|
}
|
|
return -1;
|
|
}
|
|
|
|
JGuiObject * GuiLayer::getByIndex(int index)
|
|
{
|
|
return mObjects[index];
|
|
}
|
|
|