Age | Commit message (Collapse) | Author |
|
This changes the convention back to skipping the thumbnail loading
for readSaveGameHeader. querySaveMetaInfos is the only place it
shouldn't be skipped.
|
|
This uses the one in Common.
The engine now contains a 10-bit cosine and sine table.
It used mostly for vector math.
This also allows two vector functions to be removed from vector.h.
|
|
|
|
Parallaction::_gfx is referenced by objects destroyed when
Parallaction::_input is destroyed so it cannot be destroyed first.
|
|
|
|
Also removes an unnecessary second condition check for oldAnswer
by moving that closer to its point of use.
|
|
This occurs when FreeType2 is not enabled and thus fontHeight variable
is unused.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
It's possible to get the "Fire! Fire!" timer stuck after setting
the the inquisitor doll on fire. This should work around that.
|
|
It was possible to render the game unplayable simply by looking at
Jack's cigar box while waiting for him to return to examine the
lamp. Note that this bug is only present in the DVD version. For
whatever reason, it adds a dummy location for playing the cutscene.
Applying the bugfix in the CD version actually breaks the game.
|
|
format is unavailable
|
|
|
|
|
|
This can be restored if required at a later time.
|
|
This has no functional change, but improves the consistency with the
renamed stopSound() function.
Also, minor fixes for formatting, removal of redundant comments and code
in sound class.
|
|
|
|
Since this was unused, it was causing various compiler warnings of the
set-but-not-used type. If it is needed later, this code can be restored.
|
|
Some of the debug statements in the engine compute values or sizes of
various items by pointer subtraction (which is probably not recommended;
I am not sure if this is why some of the structs were previous packed as
noted and removed by snover).
In any case, the subtractions should result in relatively small integer
values, but using these into debug() calls with printf style format
strings can cause warnings from the compiler with the format specifier
depending on the underlying pointer sizes.
To avoid these, have recast these to int. If this does cause any issues,
they should be limited to debug() value changes and thus not a
functional issue with the engine, which can be corrected by the engine
developers.
|
|
|
|
Fixes Trac#10642.
The original engine will move the dome sliders whenever the player is
dragging a dome slider to the left or right regardless of y position.
In ScummVM the dome slider position would only change to the players x
mouse position when the y value was also in the slider hotspot. This
change removes the y check by making the point to be checked always
have a y value in the hotspot rect.
The x values are also bound to the max and min value that any of the
slider hotspots can have. This allows the slider to go all the way to
the left and right even if the user has gone past the slider area
to the left or right while still holding the slider.
|
|
|
|
|
|
|
|
|
|
|
|
Fixes Trac#10619.
The thumbnail loading for the saved games was defaulting to
disabled so no thumbnail was being loaded and this caused trying
to show the thumbnail to crash scummvm.
So I have set the thumbnail pointer to be null if the thumbnail
doesn't get loaded and I've set the thumnail to not be skipped
so that thumbnails get shown when using the gui.
|
|
|
|
|
|
|
|
|
|
|
|
MOHAWK: RIVEN: Don't allow autosaving during credits
|
|
|
|
There is a brief period after the credits finish, but before the engine
closes that autosaving can happen.
Fixes Trac#10635.
|
|
|
|
|
|
|
|
|
|
Before the playtime was starting over when ScummVM was closed and
reopened instead of starting off with the playtime in the saved game.
|
|
This will make setting the playtime for the engine easier since
the current savestate stores it as a string.
This value gets set at the same time that the string playtime gets set.
|
|
|
|
|
|
Fixes Trac#10573.
|
|
Since 'filename' is 'baseName', plus a file extension, make the
'filename' buffer larger than the 'baseName' buffer. Even though
this almost certainly means that the 'filename' buffer is now
quite a bit larger than it needs to be, I don't think there's any
reason to quibble about it.
|