diff options
author | Cameron Cawley | 2019-12-31 19:19:42 +0000 |
---|---|---|
committer | Filippos Karapetis | 2020-01-11 17:34:12 +0200 |
commit | 948c555ea616821ed7c2678ad406ed5bea392339 (patch) | |
tree | 000a4566cd70bdddff6d8f4d64edfc3db9119ac5 /engines/pink | |
parent | 395f707203e8f30add0b2f8e9d8616dd12468676 (diff) | |
download | scummvm-rg350-948c555ea616821ed7c2678ad406ed5bea392339.tar.gz scummvm-rg350-948c555ea616821ed7c2678ad406ed5bea392339.tar.bz2 scummvm-rg350-948c555ea616821ed7c2678ad406ed5bea392339.zip |
ALL: Create all instances of NEResources and PEResources using new instead of on the stack
Also adapted WinCursorGroup and MacMenu to reflect this.
Diffstat (limited to 'engines/pink')
-rw-r--r-- | engines/pink/director.h | 4 | ||||
-rw-r--r-- | engines/pink/gui.cpp | 2 | ||||
-rw-r--r-- | engines/pink/pink.cpp | 13 | ||||
-rw-r--r-- | engines/pink/pink.h | 10 |
4 files changed, 16 insertions, 13 deletions
diff --git a/engines/pink/director.h b/engines/pink/director.h index 9dff49a271..a3255fc859 100644 --- a/engines/pink/director.h +++ b/engines/pink/director.h @@ -30,10 +30,6 @@ #include "graphics/macgui/macwindowmanager.h" #include "graphics/screen.h" -namespace Common { - class PEResources; -} - namespace Graphics { class MacMenu; } diff --git a/engines/pink/gui.cpp b/engines/pink/gui.cpp index 5267cbb540..9286747f63 100644 --- a/engines/pink/gui.cpp +++ b/engines/pink/gui.cpp @@ -129,7 +129,7 @@ static void menuCommandsCallback(int action, Common::U32String &, void *data) { engine->executeMenuCommand(action); } -void PinkEngine::initMenu(Common::PEResources &exeResources) { +void PinkEngine::initMenu(Common::PEResources *exeResources) { _director->getWndManager().setEnginePauseCallback(this, &pauseEngine); _menu = Graphics::MacMenu::createMenuFromPEexe(exeResources, &_director->getWndManager()); diff --git a/engines/pink/pink.cpp b/engines/pink/pink.cpp index 2852348a72..afff44d718 100644 --- a/engines/pink/pink.cpp +++ b/engines/pink/pink.cpp @@ -40,7 +40,7 @@ namespace Pink { PinkEngine::PinkEngine(OSystem *system, const ADGameDescription *desc) - : Engine(system), _console(nullptr), _rnd("pink"), + : Engine(system), _console(nullptr), _rnd("pink"), _exeResources(nullptr), _desc(desc), _bro(nullptr), _menu(nullptr), _actor(nullptr), _module(nullptr), _director(nullptr), _pdaMgr(this) { @@ -56,6 +56,7 @@ PinkEngine::PinkEngine(OSystem *system, const ADGameDescription *desc) PinkEngine::~PinkEngine() { delete _console; + delete _exeResources; delete _bro; _pdaMgr.close(); for (uint i = 0; i < _modules.size(); ++i) { @@ -72,16 +73,16 @@ Common::Error PinkEngine::init() { debugC(10, kPinkDebugGeneral, "PinkEngine init"); initGraphics(640, 480); - Common::PEResources exeResources; + _exeResources = new Common::PEResources(); Common::String fileName = isPeril() ? "pptp.exe" : "hpp.exe"; - if (!exeResources.loadFromEXE(fileName)) { + if (!_exeResources->loadFromEXE(fileName)) { return Common::kNoGameDataFoundError; } _console = new Console(this); _director = new Director(); - initMenu(exeResources); + initMenu(_exeResources); Common::String orbName; Common::String broName; @@ -96,7 +97,7 @@ Common::Error PinkEngine::init() { if (!_orb.open(orbName) || (_bro && !_bro->open(broName) && _orb.getTimestamp() == _bro->getTimestamp())) return Common::kNoGameDataFoundError; - if (!loadCursors(exeResources)) + if (!loadCursors(_exeResources)) return Common::kNoGameDataFoundError; setCursor(kLoadingCursor); @@ -233,7 +234,7 @@ bool PinkEngine::checkValueOfVariable(Common::String &variable, Common::String & return _variables[variable] == value; } -bool PinkEngine::loadCursors(Common::PEResources &exeResources) { +bool PinkEngine::loadCursors(Common::PEResources *exeResources) { bool isPokus = !isPeril(); _cursors.reserve(kCursorsCount); diff --git a/engines/pink/pink.h b/engines/pink/pink.h index cfc7190399..9c32abae08 100644 --- a/engines/pink/pink.h +++ b/engines/pink/pink.h @@ -62,6 +62,10 @@ struct ADGameDescription; +namespace Common { + class PEResources; +} + namespace Graphics { class MacMenu; } @@ -132,9 +136,9 @@ public: private: Common::Error init(); - void initMenu(Common::PEResources &exeResources); + void initMenu(Common::PEResources *exeResources); - bool loadCursors(Common::PEResources &exeResources); + bool loadCursors(Common::PEResources *exeResources); void initModule(const Common::String &moduleName, const Common::String &pageName, Archive *saveFile); void addModule(const Common::String &moduleName); @@ -148,6 +152,8 @@ private: Common::String _nextModule; Common::String _nextPage; + Common::PEResources *_exeResources; + OrbFile _orb; BroFile *_bro; |