aboutsummaryrefslogtreecommitdiff
path: root/engines/gob
AgeCommit message (Collapse)Author
2016-01-02GOB: Remove unnecessary includes and forward-declarationsSven Hesse
2015-09-02GOB: Make the currently playing CD track all-uppercaseSven Hesse
This is what the original game does too, and the scripts query for all-uppercase CD track names. Interestingly, this being wrong has been mostly inconsequential. It does, however, manifest in one bug in the underwater wreck level of Gobliins 2. After calling the Moray Eel by lighting the lamp with Winkle, speaking to said eel has the scripts wait for the currently playing background track to end before playing the voice lines, instead of ending the audio track forcefully. The track is only about a minute long, so it's "only" annoying, not a game-stopper. The scripts also try to compare the CD track name with some different, all-uppercase names, so this is possibly relevant in some other places as well. No such bug report exists at the moment, though. See also the forum post with the bug report: http://forums.scummvm.org/viewtopic.php?p=81733#81733 A somewhat related bug report is #2999 "GOB2 : Moray Eel Game Freeze (Wreck Scene)" (<http://sourceforge.net/p/scummvm/bugs/2999/>). At that time, the script expression parser still did a stricmp for the NEQ expression (which is wrong), and that made the bug not trigger. Commit 5c58b9a3a4a8848474aba18c02c0da997fb138a2, which fixed that incorrect NEQ behaviour, then made this bug here appear.
2015-08-03GOB: Remove a useless call to Variables::getAddressOff8()Sven Hesse
This is a left-over vestige of when the gob engine recorded the size of each variable currently in use, because it byte-swapped all variables on save/load depending on the machine ScummVM ran on. The gob engine doesn't do that all anymore (instead, the variables are always stored in the endianness of the original game), so getAddressOff8() doesn't store "this is an 8-bit variable" anymore, making the call a NOP. This fixes Coverity Scan issue #1267142.
2015-07-19GOB: Get rid of unused Audio::mixer referencesMax Horn
Also remove the unused AdLib::_rate member variable.
2015-07-07AUDIO: Remove all AudioStream access to OPLMatthew Hoops
2015-07-07GOB: Implement custom AdLib volume controlWalter van Niftrik
2015-07-07GOB: Use the built-in OPL timerMatthew Hoops
2015-07-07AUDIO: Remove the sample rate configuration from the OPL codeMatthew Hoops
2015-05-07GOB: Make coordinate parameters in Surface::fillRect() signedSven Hesse
And clip to [0, width), [0, height) before drawing. This fixes bug #6864, which is a regression I introduced in 51fd528fe56e00466255d54e1e71b19f34729bfd when I changed all the drawing code to use the Surface class. I thought that having unsigned coordinates makes sense, but for some reason, Fascination sets _destSpriteX (which maps to left in fillRect()) to -1, expecting the drawing code to clip.
2014-10-19GOB: Remove useless memcpy()Sven Hesse
Happened while removing Draw::_vgaSmallPalette in 0f9b1364c. Fixes Coverity issue #1230309.
2014-10-04GOB: Update list of files that contain translatable stringsThierry Crozat
2014-07-08GOB: Remove duplicate entries for Ween DOSSven Hesse
Seems like that happened when I sorted them by platform after splitting up the big detection table in 07b17f711. This fixes bug #6676.
2014-05-27ALL: Rename Debugger::DCmd_Register to Debugger::registerCmd.Johannes Schickel
2014-05-27ALL: Rename Debugger::DebugPrintf to Debugger::debugPrintf.Johannes Schickel
2014-03-16GOB: Calculate _totResStart even if no resources followSven Hesse
It's also used as a base offset for TOT strings. This fixes a crash in the Amiga version of Bargon Attack.
2014-03-16GOB: Remove the last remnants of Draw::_vgaSmallPaletteSven Hesse
Draw::_vgaSmallPalette was supposed to be a 16-color VGA palette / EGA palette, but we've been using the first 16 entries of Draw::_vgaPalette for that for ages now. The only parts where we didn't were codepaths never actually taken in the gob games, so for the sake of clarity (and fixing Coverity issues), this commit removes those as well. Fixes Coverity issues 1192648 and 1192649.
2014-02-28IMAGE: Move all ImageDecoders to image/Matthew Hoops
2014-02-18GOB: Make GPL headers consistent in themselves.Johannes Schickel
2014-02-09GOB: Fascination - Reduce the scope of a couple of variablesStrangerke
2014-01-31GOB: Correct the language field for CroustibatSven Hesse
2014-01-31GOB: Add a game type and detection table for CroustibatSven Hesse
2014-01-31GOB: Move getTotTextItemPart() from Inter_v3 to Inter_v2Sven Hesse
2013-11-24BUILD: Remove need for engine.mk in each engine directory.D G Turner
Each engine now only has to provide a single configure.engine file adding the engine into the configure script, which then produces the required other files automatically.
2013-11-24BUILD: Remove need for engine-plugin.h in engines.D G Turner
This is now generated automatically by the configure script from the engine directory names.
2013-11-24BUILD: Split engines/plugins_table header down to a file per engine.D G Turner
This is the third and final commit enabling fully pluggable engines. Now providing an engine folder contains a configure.engine, engine.mk and engine-plugin.h file, it will be picked up automatically by the configure script.
2013-11-24BUILD: Split engines.mk down to a single file per engine.D G Turner
This is the second part of allowing engines to be added dynamically. Each folder in engines/ which must contain a file named "engine.mk" containing the make definitions for that engine.
2013-11-24BUILD: Split configure.engines down to a single file per engine.D G Turner
This is the first part of allowing engines to be added dynamically. They are placed into a folder in engines/ which must contain a file named "configure.engine" to add the engine, which is pulled into the top level configure script automatically.
2013-08-12I18N: Move specification of engine specific files to enginedir/POTFILES.Johannes Schickel
This allows to keep the engines to specfiy the files for translation close to the engine sources itself. Thanks to criezy for his suggestion on this approach.
2013-08-08COMMON: Rename ConfigFile to INIFile.Johannes Schickel
This clears up that 'ConfigFile' is actually a class handling only INI-files.
2013-08-03GOB: Take advantage of Surface::getPixels.Johannes Schickel
2013-08-03GOB: Prefer getBasePtr over direct Surface::pixels access.Johannes Schickel
2013-06-30GOB: Add hypothetical 32bpp supportSven Hesse
2013-05-02COMMON: Change kPlatformPC to kPlatformDOSMatthew Hoops
"PC" was very ambiguous and now it matches what we show in the GUI. This also corrects sword2's platform to Windows.
2013-04-19Merge pull request #322 from fingolfin/fix-clang-warningsJohannes Schickel
Fix clang warnings
2013-04-19GOB: Add a call to Inter::animPalette() in the hotspot checkerSven Hesse
The original has the call there too, as confirmed using the disasm of the gob3 binary. This fixes bug #3611208.
2013-04-18ENGINES: Remove a bunch of unused private member variablesMax Horn
All instances uncovered by clang warnings.
2013-04-15GOB: Fix a misplaced continue / fallthroughSven Hesse
2013-04-15GOB: Change strcpy/strcat to strlcpy/strlcat for safetySven Hesse
2013-04-15GOB: Change strcpy to strlcpy for safetySven Hesse
2013-04-15GOB: Add another sanity checkSven Hesse
2013-04-15GOB: Add a sanity checkSven Hesse
2013-04-15GOB: Fix another potential array overrunSven Hesse
2013-04-15GOB: Fix a potential array overrunSven Hesse
2013-04-15GOB: Fix a leakSven Hesse
2013-04-15GOB: Correct a delete/delete[]-mismatchSven Hesse
2013-04-15GOB: Remove unused member variable in SaveLoad_v6Sven Hesse
2013-04-15GOB: Remove unused member variables in SaveLoad_FascinationSven Hesse
2013-04-08GOB: Add another Spanish DOS version of GeishaSven Hesse
As supplied by alestedx in bug report #3610207
2013-03-20GOB: Plug a leakSven Hesse
2013-03-20GOB: iterator++ -> ++iteratorSven Hesse