aboutsummaryrefslogtreecommitdiff
path: root/engines
AgeCommit message (Collapse)Author
2017-09-23SCI32: Remove unnecessary use of strncmpColin Snover
One of the strings is a static string of the same size as the size passed to strncmp so there is no point in not just using strcmp.
2017-09-23SCI32: Replace magic number with kAutoSaveId in QFG4 save codeColin Snover
2017-09-23SCI32: Support RAMA's custom save games through kFileIOColin Snover
2017-09-23SCI: Split save game metadata writing to separate functionColin Snover
RAMA has its own custom save game format that game scripts write, but we still want to be able to use these save game files from the ScummVM launcher, so the metadata has to be able to be written separately from the rest of the game saving.
2017-09-23SCI: Use reference instead of pointer for required out-data in metadata save ↵Colin Snover
function
2017-09-23SCI: Clean-ups to file handling codeColin Snover
Preparations for adding support for RAMA's custom save game format.
2017-09-23SCI32: Remove unused shrink-to-fit function for string arraysColin Snover
2017-09-23SCI32: Fix size for dereferencing non-raw SCI32 arraysColin Snover
The maxSize given when generating a SegmentRef is supposed to be the maximum size of raw data that can be held inside the offset- part of a reg_t, not the entire size including the segment. This fixes a buffer overflow in "Inside the Chest", which still does not work, but at least doesn't cause heap overflows anymore with this change.
2017-09-23TITANIC: DE: Merged German NPC script subclasses back inPaul Gilbert
There ended up not being enough custom German code in the scripts to warrant having sub-classes that reimplemented entire methods
2017-09-23TITANIC: DE: Updates for SuccUBusScriptPaul Gilbert
2017-09-23TITANIC: DE: Added changes to MaitreDScriptPaul Gilbert
2017-09-23TITANIC: DE: Added changes for LiftbotScriptPaul Gilbert
2017-09-23TITANIC: DE: Adding remaining DoorbotScript changesPaul Gilbert
2017-09-23TITANIC: DE: Adding changes to DoorbotScriptPaul Gilbert
2017-09-22TITANIC: DE: More BellbotScript changesPaul Gilbert
2017-09-22TITANIC: DE: Adding new BellbotScript logicPaul Gilbert
2017-09-22TITANIC: DE: Added translations for BarbotScript::processPaul Gilbert
2017-09-22ALL: Specify the DisposeAfterUse constructor argument for dynamic memory ↵Bastien Bouclet
write streams
2017-09-22ZVISION: Fix leaking save / thumbnail MemoryWriteStreamDynamic buffersBastien Bouclet
2017-09-22SWORD25: Fix leaking the lua state MemoryWriteStreamDynamic bufferBastien Bouclet
2017-09-22SWORD25: Fix leaking the save thumbnail MemoryWriteStreamDynamicBastien Bouclet
2017-09-22SWORD1: Fix leaking the save thumbnail MemoryWriteStreamDynamic bufferBastien Bouclet
2017-09-22GNAP: Fix leaking the save thumbnail MemoryWriteStreamDynamic bufferBastien Bouclet
2017-09-22FULLPIPE: Fix leaking the save MemoryWriteStreamDynamic bufferBastien Bouclet
2017-09-22DM: Fix leaking the save thumbnail MemoryWriteStreamDynamic bufferBastien Bouclet
2017-09-22XEEN: Change OutFile not to subclass MemoryWriteStreamDynamicBastien Bouclet
Also fix leaking the MemoryWriteStreamDynamic buffer storage
2017-09-22XEEN: Change SavesManager to use a map of pointers to MemoryWriteStreamDynamicBastien Bouclet
Our implementation of HashMap does not allow value types without a zero-argument constructor.
2017-09-21TITANIC: DE: Add miscellaneous missed sound translationsPaul Gilbert
2017-09-21TITANIC: DE: More sound translationsPaul Gilbert
2017-09-20TITANIC: DE: Adding sound translationsPaul Gilbert
2017-09-20TITANIC: DE: Adding sound translationsPaul Gilbert
2017-09-20CGE2: Clean up SearchMan after fallbackDetectWillem Jan Palenstijn
2017-09-20CGE: Clean up SearchMan after fallbackDetectWillem Jan Palenstijn
2017-09-20SLUDGE: Skip SearchMan detour in fallback detectionWillem Jan Palenstijn
2017-09-20DIRECTOR: Skip SearchMan detour in fallback detectionWillem Jan Palenstijn
2017-09-20SCI: Fix detection of end of audio map & entry size in SCI32Colin Snover
Torin RU map 38140 has an unusual terminator entry; instead of a normal terminating entry of 11 FFs, its terminating entry is 03 FF FF FF FF C4 36 01 FF FF FF. So, two changes are made: 1. The end-of-map check is now the same as in SSCI1.1+ and only checks that the final byte of the Audio36 tuple is 0xFF, instead of the entire tuple; 2. The unneeded entry size heuristic has been turned off for all SCI32 games. A quick check of the English versions of LB2CD, EQ1CD, SQ4CD, and KQ6CD, as well as all English SCI32 games, indicates that this approach seems to be working correctly. Fixes Trac#10188.
2017-09-20TITANIC: DE: Add translations for playGlobalSound callsPaul Gilbert
2017-09-20TITANIC: DE: Add translations for loadSound callsPaul Gilbert
2017-09-20MOHAWK: Riven: Move the water effect to its own classBastien Bouclet
Fixes the per frame scripts being leaked. Fixes #10215.
2017-09-20MOHAWK: Riven: Plug memory leak when savingBastien Bouclet
Fixes #10216.
2017-09-19TITANIC: DE: Adding German translationsPaul Gilbert
2017-09-19TITANIC: DE: Sound changes for BombPaul Gilbert
2017-09-19SCI32: Support RAMA's single sound effects volumeColin Snover
The original installer copied RESOURCE.SFX to the hard drive so there was only one RESOURCE.SFX on CD 1. Instead of requiring users to create duplicates, just use the single RESOURCE.SFX if it exists.
2017-09-19SCI32: Fix inconsistent patch instruction commentsColin Snover
I missed these lines when going through things earlier.
2017-09-19SCI32: Put superclass address in r_acc for SCI3 super callsColin Snover
This fixes a problem in Lighthouse 2.0a where the mini-sub would fail to start playing the animation of the shipwreck when clicking on the throttle. In SSCI, in SCI3 only, r_acc was (inadvertently?) set to the superclass object ID whenever a super call was made. This happened because OP_super would call to get the superclass object ID, the calling conventions of the compiler put this return value into EAX, and then the PMachine message processing code put whatever was in EAX into r_acc before each message was processed. In the game code, there are a sequence of steps that look like this: * First, throttle::doVerb is called when throttle is clicked on; * Which calls getRobot::doit to tell the shipwreck robot to start playing; * Which calls wreckBot::init to reset the Robot for the animation; * Which calls Hiliter::hotVerbs(0) to remove cursor hotspots; * Which calls Hiliter::dispose to clean up since it is not used; * Which causes Hiliter::verbList to get set to 0. * Later, verbList is loaded into r_acc, and it is still 0; * Then, Hiliter::dispose makes a super call to Obj::dispose; * Then, Obj::dispose does nothing except call kDisposeClone, which does not mutate r_acc, so r_acc is still 0 from verbList; * Then we return back through 5 calls to throttle::doVerb; * Then throttle::doVerb checks that r_acc is non-zero, and if so, adds wreckBot to theDoits global, allowing the animation to occur. In ScummVM, without setting r_acc in the super call, the non-zero check failed and the wreckBot didn't get put into theDoits, so the entire sequence fell apart. In SSCI, the non-zero check happened to succeed because the Obj::dispose super call put the Obj class into the accumulator. So now we do that too, and now Lighthouse 2.0a works here. Earlier versions of SSCI used EAX for other things in between the OP_super call and the message processing, so would set r_acc from different data, so this change does not apply to those versions.
2017-09-19SCI: Allow multi-step tracing with step-over in debuggerColin Snover
2017-09-19SCI32: Fix GfxFrameout::addPlane from causing possible leaksColin Snover
2017-09-19SCI32: Remove never-read default valuesColin Snover
2017-09-19SCI32: Add audio dump debugger commandColin Snover
2017-09-19SCI32: Don't warp the mouse twice when its position has been restrictedColin Snover