aboutsummaryrefslogtreecommitdiff
path: root/engines/agi
AgeCommit message (Collapse)Author
2012-02-20COMMON: Move isFoo functions to namespace Common, add doxygen commentsMax Horn
2012-02-15JANITORIAL: Fix missing whitespace in pointer castTarek Soliman
find -name '*.h' -or -name '*.cpp' | xargs sed -r -i 's@\(([A-Za-z0-9]+)\*\)@(\1 *)@g' This seems to have caught some params as well which is not undesirable IMO. It also caught some strings containing this which is undesirable so I excluded them manually. (engines/sci/engine/kernel_tables.h)
2012-02-15JANITORIAL: Fix whitespace in pointer template argTarek Soliman
2012-02-15ALL: Avoid using is* macros from ctype.hMax Horn
On some systems, passing signed chars to macros like isspace() etc. lead to a runtime error. Hence, mark these macros as forbidden by default, and introduce otherwise equivalent alternatives for them.
2012-01-07AGI: Fix bug #3451122 - "AGI-FANMADE: Nick's Quest hangs ScummVM upon starting"Filippos Karapetis
2012-01-06AGI: Fix stuck next button when switching back to predictive modeWillem Jan Palenstijn
2012-01-06AGI: Clean up predictive matchingWillem Jan Palenstijn
Specifically: * Don't enable the 'next' button in predictive mode when we don't have a full match. Doing this didn't make sense since you'd iterate over a seemingly arbitrary set of completions instead of all possible ones. * Do only a single binary search. * Fix the width of the mode button for mouse press detections.
2012-01-06AGI: Fix ignoring some exact matches in predictive inputWillem Jan Palenstijn
The matcher now always tries an exact match before trying an inexact one. Together with 41ba2433f, this fixes bug #3470080.
2012-01-06AGI: Fix predictive input binary searchWillem Jan Palenstijn
Note that there are still issues with this input mode. If the binary search hits a prefix match before finding the full match, it won't recognize the full match. (Example: typing "buy" will show "buz" because it's aiming for "buzzard" without realizing "buy" exists.)
2011-12-18Merge pull request #131 from digitall/goto_considered_harmfulDavid Turner
Goto Considered Harmful... The following commits should improve the ScummVM code structure by reducing the number of gotos used in various engine code. They should implement identical functionality, but without using goto and without the result being less readable/maintainable than the version with goto.
2011-12-13Merge pull request #137 from fingolfin/various-cleanupFilippos Karapetis
Various cleanup
2011-12-13AGI: Fix bug #3095169: AGI inventory itemsEugene Sandulenko
2011-12-12AGI: Add FIXME to cmdCallV1Max Horn
2011-12-12AGI: Remove two unused pure virtual methodsMax Horn
This avoid warnings, as these methods were being hidden by methods in subclasses with differing signatures.
2011-12-10AGI: Replace usage of 'goto'.D G Turner
2011-11-27JANITORIAL: Remove extra semicolons.Christoph Mallon
2011-11-16LAUNCHER: Introduce GUIO0() in order to replace GUIO1(GUIO_NONE)Strangerke
2011-11-16AGI: Change two local SharedPtr uses to ScopedPtr.Johannes Schickel
I took the opportunity to add two CHECKMEs to the code, which seems to load a file into memory for no real apparent reason.
2011-11-03AGI: Commented out if statements that always evaluate to trueFilippos Karapetis
2011-11-03AGI: Commented out if statements that always evaluate to trueFilippos Karapetis
2011-11-02AGI: Fix warningsEugene Sandulenko
2011-11-02AGI: Fix buffer overflowEugene Sandulenko
2011-10-23AD: Switched rest of the engines to new GUIOEugene Sandulenko
2011-10-16AGI: Refine commit #69c705a to always pick the longest matching phraseFilippos Karapetis
Thanks to wjp for his observation in the original commit
2011-10-16AGI: Fixed bug #3424066 - "LSL1 AGI: Password Glitch"Filippos Karapetis
We should not stop looking when a partial sentence match is found, as a better match might exist later on. In this case, there were two matching sentences, "Ken" (which is wrong in this case) and "Ken sent me" (which is correct, but was never reached as the partial match was returned first)
2011-10-10AGI: Fixed bug #3420859 - "AGI: SQ0 - Scummvm Crash"Filippos Karapetis
The length requested in this case is 250, which overflows a char variable
2011-09-26AGI: Fixed several incorrect substitutions of "game" with "_game"Filippos Karapetis
2011-09-26AGI: Fix styleWillem Jan Palenstijn
2011-09-26AGI: Removed leftover dead codeFilippos Karapetis
2011-09-26AGI: Fixed invalid memory writes in wordWrapString()Filippos Karapetis
2011-09-26AGI: Fixed the line changing code, and added EOL checksFilippos Karapetis
The extra checks make sure that there isn't an extra line added at the end of the string. They're added as a precautionary measure
2011-09-26AGI: Rewrote cmdVersion() to use Common::StringFilippos Karapetis
This simplifies the code and avoids heap corruption because of the long version string
2011-09-26AGI: Cleanup, added some comments, removed an obsolete FIXMEFilippos Karapetis
2011-09-26AGI: Add FIXME for wordwrap buffer overflowWillem Jan Palenstijn
2011-09-26AGI: Clean up save/load String usageWillem Jan Palenstijn
2011-09-26AGI: Fixed the description of KQ2 Amiga (bug #3191496)Filippos Karapetis
2011-09-26AGI: Fixed bug #3196882 - "KQ3: PCSpeaker affects render mode"Filippos Karapetis
2011-09-26AGI: Marked 4070342 as a workaroundFilippos Karapetis
2011-09-26AGI: Fix for bug #3018770 - "AGI: PQ1: Flickering newspaper"Filippos Karapetis
2011-09-26AGI: Fixed the actual cause of bug #3295652, and partially fixed bug #3080415Filippos Karapetis
2011-09-26AGI: doUpdate() already calls updateScreen()Filippos Karapetis
2011-09-26AGI: Fixed bug #3074570 - "AGI LSL1: TAB stops working after restart"Filippos Karapetis
Applied eriktorbjorn's patch from that bug tracker item (slightly modified), which is what NAGI does, and which fixes restarting in LSL1 and PQ1 (bug #2823762), and other AGI games that do not reset the controller keys when restarting.
2011-09-26AGI: Fix bug #3080041 - "AGI Mother Goose: White bar in intro"Filippos Karapetis
2011-09-25AGI: Fix compilationFilippos Karapetis
2011-09-25SCI: Added a workaround for bug #3295652 - "AGI: (Fan) SQ0: Animation ↵Filippos Karapetis
overdraws some of the text window" and removed some duplicate code
2011-09-25AGI: Switched to Common::String in the save/load codeFilippos Karapetis
2011-09-25AGI: Rewrite getSavegameFilename() so that it doesn't try to return a ↵Filippos Karapetis
pointer to a local variable
2011-09-25AGI: Fixed EOF in text.cppFilippos Karapetis
2011-09-25AGI: Fixed bug #3292784 - "AGI: SQ2: Name forgotten after game restart"Filippos Karapetis
The string buffer should not be cleared on game restart, as game scripts depend on it being maintained unchanged during restart.
2011-09-25AGI: Bugfix for script bug #3116541 - "AGI FAN Beyond the Titanic 2: Crash ↵Filippos Karapetis
on Item Viewing"