aboutsummaryrefslogtreecommitdiff
path: root/engines/sci/engine
diff options
context:
space:
mode:
authorsluicebox2019-01-30 19:41:10 -0800
committerFilippos Karapetis2019-02-03 15:40:01 +0200
commita0b319646c105313cfe218931bb5e4eff4b1ee5e (patch)
tree53d5baa05653e1d721d993439730ec107f012f4c /engines/sci/engine
parenta7f1230980674397486af0c8ec4057d83f3ff0dd (diff)
downloadscummvm-rg350-a0b319646c105313cfe218931bb5e4eff4b1ee5e.tar.gz
scummvm-rg350-a0b319646c105313cfe218931bb5e4eff4b1ee5e.tar.bz2
scummvm-rg350-a0b319646c105313cfe218931bb5e4eff4b1ee5e.zip
SCI: Fix ECO1CD giving Superfluous the oily shell
Fixes a script bug in the CD version, bug #10881
Diffstat (limited to 'engines/sci/engine')
-rw-r--r--engines/sci/engine/script_patches.cpp42
1 files changed, 40 insertions, 2 deletions
diff --git a/engines/sci/engine/script_patches.cpp b/engines/sci/engine/script_patches.cpp
index 4d941af938..c657fee064 100644
--- a/engines/sci/engine/script_patches.cpp
+++ b/engines/sci/engine/script_patches.cpp
@@ -548,9 +548,47 @@ static const uint16 ecoquest1PatchStayAndHelp[] = {
PATCH_END
};
-// script, description, signature patch
+// Giving the oily shell to Superfluous when he's out of the mask runs the
+// wrong animation and skips messages in the CD version. Sierra modified
+// getInOilyShell for the CD version by adding a new state to the beginning
+// but forgot to increment the state numbers passed to self:changeState to
+// their new values, causing the script to change to the wrong states.
+//
+// We fix this by incrementing the state numbers passed to self:changeState.
+//
+// Applies to: PC CD
+// Responsible method: getInOilyShell:changeState
+// Fixes bug #10881
+static const uint16 ecoquest1SignatureGiveOilyShell[] = {
+ 0x30, SIG_UINT16(0x000a), // bnt 000a
+ 0x38, SIG_UINT16(0x0090), // pushi changeState [ hard coded for CD ]
+ 0x78, // push1
+ 0x7a, // push2 [ state 2 ]
+ 0x54, SIG_MAGICDWORD, 0x06, // self 06
+ 0x32, SIG_UINT16(0x0195), // jmp 0195
+ SIG_ADDTOOFFSET(+209),
+ 0x39, 0x08, // pushi 08 [ state 8 ]
+ SIG_ADDTOOFFSET(+16),
+ 0x39, 0x0b, // pushi 0b [ state 11 ]
+ SIG_END
+};
+
+static const uint16 ecoquest1PatchGiveOilyShell[] = {
+ 0x31, 0x0b, // bnt 0b
+ 0x38, PATCH_UINT16(0x0090), // pushi changeState [ hard coded for CD ]
+ 0x78, // push1
+ 0x39, 0x03, // pushi 03 [ state 3 ]
+ PATCH_ADDTOOFFSET(+214),
+ 0x39, 0x09, // pushi 09 [ state 9 ]
+ PATCH_ADDTOOFFSET(+16),
+ 0x39, 0x0c, // pushi 0c [ state 12 ]
+ PATCH_END
+};
+
+// script, description, signature patch
static const SciScriptPatcherEntry ecoquest1Signatures[] = {
- { true, 660, "CD: bad messagebox and freeze", 1, ecoquest1SignatureStayAndHelp, ecoquest1PatchStayAndHelp },
+ { true, 160, "CD: give superfluous oily shell", 1, ecoquest1SignatureGiveOilyShell, ecoquest1PatchGiveOilyShell },
+ { true, 660, "CD: bad messagebox and freeze", 1, ecoquest1SignatureStayAndHelp, ecoquest1PatchStayAndHelp },
SCI_SIGNATUREENTRY_TERMINATOR
};