From d7112b5bbc17e80065534d5308f24ea0fefeedaf Mon Sep 17 00:00:00 2001 From: Willem Jan Palenstijn Date: Sun, 27 Feb 2011 15:44:12 +0100 Subject: [PATCH] SCI: Remove kDoBresen special case for SCI_VERSION_1_EGA_ONLY Confirmed with QfG2 disassembly (the only 1_EGA_ONLY game) that it behaves the same as SCI_VERSION_1_EARLY games, except for the uninitialized read already mentioned in kDoBresen. --- engines/sci/engine/kmovement.cpp | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/engines/sci/engine/kmovement.cpp b/engines/sci/engine/kmovement.cpp index bc22397c150f..392db56419ac 100644 --- a/engines/sci/engine/kmovement.cpp +++ b/engines/sci/engine/kmovement.cpp @@ -374,17 +374,10 @@ reg_t kDoBresen(EngineState *s, int argc, reg_t *argv) { writeSelectorValue(segMan, mover, SELECTOR(b_i2), mover_i2); writeSelectorValue(segMan, mover, SELECTOR(b_di), mover_di); - if (getSciVersion() == SCI_VERSION_1_EGA_ONLY) { - // We need to compare directly in here, complete may have happened during - // the current move - if ((client_x == mover_x) && (client_y == mover_y)) - invokeSelector(s, mover, SELECTOR(moveDone), argc, argv); - } - - if (getSciVersion() >= SCI_VERSION_1_EARLY) { - // This calling code here was right before the last return in - // sci1ega and got changed to this position since sci1early - // this was an uninitialized issue in sierra sci + if (getSciVersion() >= SCI_VERSION_1_EGA_ONLY) { + // In sci1egaonly this block of code was outside of the main if, + // but client_x/client_y aren't set there, so it was an + // uninitialized read in SSCI. (This issue was fixed in sci1early.) if (handleMoveCount) writeSelectorValue(segMan, mover, SELECTOR(b_movCnt), mover_moveCnt); // We need to compare directly in here, complete may have happened during