mirror of
https://github.com/OpenMW/openmw.git
synced 2025-02-21 13:09:42 +00:00
Use QPointer to detect object existence, less verbose debug messages
This commit is contained in:
parent
b84e41bd27
commit
d62ddc002e
2 changed files with 31 additions and 16 deletions
|
@ -1,5 +1,7 @@
|
|||
#include "commands.hpp"
|
||||
|
||||
#include <QPointer>
|
||||
|
||||
#include <components/debug/debuglog.hpp>
|
||||
#include <components/esm/loadland.hpp>
|
||||
|
||||
|
@ -15,26 +17,38 @@ CSVRender::DrawTerrainSelectionCommand::DrawTerrainSelectionCommand(WorldspaceWi
|
|||
|
||||
void CSVRender::DrawTerrainSelectionCommand::redo()
|
||||
{
|
||||
if (CSVRender::WorldspaceWidget* worldspaceWidget = dynamic_cast<CSVRender::WorldspaceWidget *> (mWorldspaceWidget))
|
||||
if (mWorldspaceWidget)
|
||||
{
|
||||
if (CSVRender::TerrainShapeMode* terrainMode = dynamic_cast<CSVRender::TerrainShapeMode *> (worldspaceWidget->getEditMode()) )
|
||||
{
|
||||
terrainMode->getTerrainSelection()->update();
|
||||
return;
|
||||
}
|
||||
if (CSVRender::TerrainShapeMode* terrainMode = dynamic_cast<CSVRender::TerrainShapeMode *> (mWorldspaceWidget->getEditMode()) )
|
||||
{
|
||||
terrainMode->getTerrainSelection()->update();
|
||||
return;
|
||||
}
|
||||
else
|
||||
{
|
||||
Log(Debug::Verbose) << "Can't update terrain selection in current EditMode";
|
||||
return;
|
||||
}
|
||||
}
|
||||
Log(Debug::Warning) << "Error in redoing terrain selection";
|
||||
else
|
||||
Log(Debug::Verbose) << "Can't update terrain selection, no WorldspaceWidget found!";
|
||||
}
|
||||
|
||||
void CSVRender::DrawTerrainSelectionCommand::undo()
|
||||
{
|
||||
if (CSVRender::WorldspaceWidget* worldspaceWidget = dynamic_cast<CSVRender::WorldspaceWidget *> (mWorldspaceWidget))
|
||||
if (mWorldspaceWidget)
|
||||
{
|
||||
if (CSVRender::TerrainShapeMode* terrainMode = dynamic_cast<CSVRender::TerrainShapeMode *> (worldspaceWidget->getEditMode()) )
|
||||
{
|
||||
terrainMode->getTerrainSelection()->update();
|
||||
return;
|
||||
}
|
||||
if (CSVRender::TerrainShapeMode* terrainMode = dynamic_cast<CSVRender::TerrainShapeMode *> (mWorldspaceWidget->getEditMode()) )
|
||||
{
|
||||
terrainMode->getTerrainSelection()->update();
|
||||
return;
|
||||
}
|
||||
else
|
||||
{
|
||||
Log(Debug::Verbose) << "Can't undo terrain selection in current EditMode";
|
||||
return;
|
||||
}
|
||||
}
|
||||
Log(Debug::Warning) << "Error in undoing terrain selection";
|
||||
else
|
||||
Log(Debug::Verbose) << "Can't undo terrain selection, no WorldspaceWidget found!";
|
||||
}
|
||||
|
|
|
@ -1,6 +1,8 @@
|
|||
#ifndef CSV_RENDER_COMMANDS_HPP
|
||||
#define CSV_RENDER_COMMANDS_HPP
|
||||
|
||||
#include <QPointer>
|
||||
|
||||
#include <QUndoCommand>
|
||||
|
||||
#include "worldspacewidget.hpp"
|
||||
|
@ -23,10 +25,9 @@ namespace CSVRender
|
|||
*/
|
||||
class DrawTerrainSelectionCommand : public QUndoCommand
|
||||
{
|
||||
friend class WorldspaceWidget;
|
||||
|
||||
private:
|
||||
WorldspaceWidget *mWorldspaceWidget;
|
||||
QPointer<WorldspaceWidget> mWorldspaceWidget;
|
||||
|
||||
public:
|
||||
DrawTerrainSelectionCommand(WorldspaceWidget* worldspaceWidget, QUndoCommand* parent = nullptr);
|
||||
|
|
Loading…
Reference in a new issue