diff options
| author | Eugene Sandulenko | 2016-09-18 00:11:21 +0200 |
|---|---|---|
| committer | Eugene Sandulenko | 2016-09-18 00:16:29 +0200 |
| commit | d9457c2f07f22dae83b6702fa7b616aa41696a7f (patch) | |
| tree | a54de8c8860f6228b92778be9e2eab9e96c3e890 | |
| parent | f67bbd9f8c14133dec61ff88d314d381b1014272 (diff) | |
| download | scummvm-rg350-d9457c2f07f22dae83b6702fa7b616aa41696a7f.tar.gz scummvm-rg350-d9457c2f07f22dae83b6702fa7b616aa41696a7f.tar.bz2 scummvm-rg350-d9457c2f07f22dae83b6702fa7b616aa41696a7f.zip | |
FULLPIPE: Remove workaround and follow the original in StaticANIObject copy constructor
| -rw-r--r-- | engines/fullpipe/statics.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/engines/fullpipe/statics.cpp b/engines/fullpipe/statics.cpp index c18affb567..620cb50788 100644 --- a/engines/fullpipe/statics.cpp +++ b/engines/fullpipe/statics.cpp @@ -191,9 +191,9 @@ StaticANIObject::StaticANIObject(StaticANIObject *src) : GameObject(src) { Movement *newmov; if (src->_movements[i]->_currMovement) { - // WORKAROUND: Original uses weird construction here: - // new Movement(getMovementById(src->getMovementIdById(mov->_id)), this); - newmov = new Movement(src->getMovementById(src->getMovementIdById(src->_movements[i]->_id)), this); + // This is weird code. Logically it should be + // newmov = new Movement(src->getMovementById(src->getMovementIdById(src->_movements[i]->_id)), this); + newmov = new Movement(getMovementById(src->getMovementIdById(src->_movements[i]->_id)), this); newmov->_id = src->_movements[i]->_id; } else { newmov = new Movement(src->_movements[i], 0, -1, this); |