diff options
| author | Andrei Prykhodko | 2018-06-30 23:49:27 +0300 |
|---|---|---|
| committer | Andrei Prykhodko | 2018-06-30 23:49:27 +0300 |
| commit | 0a6993a09b9efd9cd17243e51aa64522bec78b4e (patch) | |
| tree | 1344494d5961dc97e6149e8cd3eba543df980a0a /engines/pink/objects | |
| parent | 22f6c6639381adf3dfad45ba1cf430dd182e6322 (diff) | |
| download | scummvm-rg350-0a6993a09b9efd9cd17243e51aa64522bec78b4e.tar.gz scummvm-rg350-0a6993a09b9efd9cd17243e51aa64522bec78b4e.tar.bz2 scummvm-rg350-0a6993a09b9efd9cd17243e51aa64522bec78b4e.zip | |
PINK: fixed uninitialized fields
Diffstat (limited to 'engines/pink/objects')
| -rw-r--r-- | engines/pink/objects/actions/action_play_with_sfx.h | 4 | ||||
| -rw-r--r-- | engines/pink/objects/actors/lead_actor.cpp | 7 | ||||
| -rw-r--r-- | engines/pink/objects/inventory.cpp | 4 | ||||
| -rw-r--r-- | engines/pink/objects/pages/game_page.cpp | 4 | ||||
| -rw-r--r-- | engines/pink/objects/sequences/seq_timer.cpp | 3 | ||||
| -rw-r--r-- | engines/pink/objects/sequences/sequence.h | 3 | ||||
| -rw-r--r-- | engines/pink/objects/sequences/sequence_item.h | 3 | ||||
| -rw-r--r-- | engines/pink/objects/walk/walk_mgr.cpp | 3 |
8 files changed, 22 insertions, 9 deletions
diff --git a/engines/pink/objects/actions/action_play_with_sfx.h b/engines/pink/objects/actions/action_play_with_sfx.h index f357c8ddb4..64a19d46cb 100644 --- a/engines/pink/objects/actions/action_play_with_sfx.h +++ b/engines/pink/objects/actions/action_play_with_sfx.h @@ -32,6 +32,8 @@ class ActionSfx; class ActionPlayWithSfx : public ActionPlay { public: + ActionPlayWithSfx() + : _isLoop(false) {} ~ActionPlayWithSfx() override; void deserialize(Archive &archive) override; @@ -45,7 +47,7 @@ protected: private: Array<ActionSfx *> _sfxArray; - uint32 _isLoop; + bool _isLoop; }; class Page; diff --git a/engines/pink/objects/actors/lead_actor.cpp b/engines/pink/objects/actors/lead_actor.cpp index 61d367e786..357d739a40 100644 --- a/engines/pink/objects/actors/lead_actor.cpp +++ b/engines/pink/objects/actors/lead_actor.cpp @@ -33,9 +33,10 @@ namespace Pink { LeadActor::LeadActor() - : _state(kReady), _nextState(kReady), _isHaveItem(false), - _recipient(nullptr), _cursorMgr(nullptr), _walkMgr(nullptr), - _sequencer(nullptr), _audioInfoMgr(this) {} + : _state(kReady), _nextState(kUndefined), _stateBeforeInventory(kUndefined), + _stateBeforePDA(kUndefined), _isHaveItem(false), _recipient(nullptr), + _cursorMgr(nullptr), _walkMgr(nullptr), _sequencer(nullptr), + _audioInfoMgr(this) {} void LeadActor::deserialize(Archive &archive) { _state = kReady; diff --git a/engines/pink/objects/inventory.cpp b/engines/pink/objects/inventory.cpp index b4c1f7db56..c99e6c6d9c 100644 --- a/engines/pink/objects/inventory.cpp +++ b/engines/pink/objects/inventory.cpp @@ -33,7 +33,9 @@ namespace Pink { InventoryMgr::InventoryMgr() - : _lead(nullptr), _item(nullptr), _isClickedOnItem(false) {} + : _lead(nullptr), _window(nullptr), _itemActor(nullptr), + _rightArrow(nullptr), _leftArrow(nullptr), _state(kIdle), + _isClickedOnItem(false) {} void InventoryItem::deserialize(Archive &archive) { NamedObject::deserialize(archive); diff --git a/engines/pink/objects/pages/game_page.cpp b/engines/pink/objects/pages/game_page.cpp index 1157d96f4e..a45fca7b7f 100644 --- a/engines/pink/objects/pages/game_page.cpp +++ b/engines/pink/objects/pages/game_page.cpp @@ -31,8 +31,8 @@ namespace Pink { GamePage::GamePage() - : _cursorMgr(nullptr), _walkMgr(nullptr), _sequencer(nullptr), - _isLoaded(false), _memFile(nullptr) {} + : _module(nullptr), _cursorMgr(nullptr), _walkMgr(nullptr), + _sequencer(nullptr), _isLoaded(false), _memFile(nullptr) {} GamePage::~GamePage() { clear(); diff --git a/engines/pink/objects/sequences/seq_timer.cpp b/engines/pink/objects/sequences/seq_timer.cpp index c1eff0765f..5342dbff4a 100644 --- a/engines/pink/objects/sequences/seq_timer.cpp +++ b/engines/pink/objects/sequences/seq_timer.cpp @@ -32,7 +32,8 @@ namespace Pink { SeqTimer::SeqTimer() - : _updatesToMessage(0) {} + : _sequencer(nullptr), _updatesToMessage(0), _period(0), + _range(0) {} void SeqTimer::deserialize(Archive &archive) { _actor = archive.readString(); diff --git a/engines/pink/objects/sequences/sequence.h b/engines/pink/objects/sequences/sequence.h index 7ebb95ce4c..b07e7508fd 100644 --- a/engines/pink/objects/sequences/sequence.h +++ b/engines/pink/objects/sequences/sequence.h @@ -72,6 +72,9 @@ protected: class SequenceAudio : public Sequence { public: + SequenceAudio() + : _leader(nullptr) {} + void deserialize(Archive &archive) override; void toConsole() override; diff --git a/engines/pink/objects/sequences/sequence_item.h b/engines/pink/objects/sequences/sequence_item.h index 4005331783..554fd73819 100644 --- a/engines/pink/objects/sequences/sequence_item.h +++ b/engines/pink/objects/sequences/sequence_item.h @@ -54,6 +54,9 @@ public: class SequenceItemLeaderAudio : public SequenceItemLeader { public: + SequenceItemLeaderAudio() + : _sample(0) {} + void deserialize(Archive &archive) override; void toConsole() override; diff --git a/engines/pink/objects/walk/walk_mgr.cpp b/engines/pink/objects/walk/walk_mgr.cpp index 6bda000eb8..21e8392875 100644 --- a/engines/pink/objects/walk/walk_mgr.cpp +++ b/engines/pink/objects/walk/walk_mgr.cpp @@ -29,7 +29,8 @@ namespace Pink { WalkMgr::WalkMgr() - : _isWalking(false), _leadActor(nullptr) {} + : _isWalking(false), _leadActor(nullptr), + _destination(nullptr) {} WalkMgr::~WalkMgr() { for (uint i = 0; i < _locations.size(); ++i) { |
