| Age | Commit message (Collapse) | Author |
|---|
|
Fixes Trac#10516.
|
|
In D'ni, after giving the white page to Atrus, or after coming without a
page, too videos of Atrus would play on top of each other after waiting
long enough for the first video to loop.
Fixes #10503.
|
|
The global variable "ending" tracks various
states that Atrus can be in.
This variable determines what video of
Atrus will be playing and if the brother
books have been destroyed.
Despite this variable being called ending
there are other endings not captured by
this variable (like the brother endings).
Also change spelling of Dini to Dni for one
usage of ActiveAge from a previous commit.
|
|
Playing the credits unloads the data required by the rest of the script.
Fixes Trac#10502.
|
|
The credit stack does not have the data required to display all the
cursors, especially the page cursors.
Fixes Trac#10501.
|
|
This global basically tracks what age/book/stack
the player is in.
It is also used to change stacks via
MohawkEngine_Myst::changeToStack() by
being an index for MystScriptParser::_stackMap
and MystScriptParser::_startCard.
E.x., instead of _global.currentAge = 2
now _global.currentAge = kMystLibrary.
Also two of the endings are considered Ages
instead of being apart of _global.ending.
|
|
Previously, the held page stage was an unsigned int 16
with values 0-13. The enum will make its state more clear.
|
|
Fixes Trac#10499.
|
|
Fixes Achenar's door animation being mispositionned on Channelwood.
Fixes Trac#10498.
|
|
Without the fall-through, the function would unconditionally dereference
an uninitialized pointer.
|
|
Fall-throughs are used as a way to handle optional parameters in this
function.
|
|
Changing the background sound when switching cards could stop the effect
sound immediately after it started.
Fixes Trac#10493.
|
|
|
|
|
|
Fixes the sound receiver display stopping for a little while
every second or so when keeping a direction button pressed.
|
|
In the original the sound reciever would provide a hint of the
source sound direction when the user was less than 5 degrees
away. In ScummVM it was less than or equal to 5 degrees.
Now it matches the behavior of the original.
|
|
This slows down the Selentic age sound receiver spin rate
when fully depressing either of the turn buttons.
This is a deviation from the original and improves
the users experience.
In the original when the button is fully depressed
the receiver turns so fast that it is impossible
to predict what angle you full be at when you let go.
|
|
|
|
Fixes #Trac10486.
For 12 Hr format versions, in ScummVM when the hour was set to
what should be 12:XX it was displayed as 0:XX.
This differs with the original English versions which displays
the 12:xx.
I verified this fix in both Myst ME and Myst English CD.
|
|
Add delays to simulate running at a lower framerate.
|
|
Fixes #10489.
|
|
Fixes #10485.
|
|
* Keep playing the previously running background sound while playing the
flyby.
* Don't play the flyby after loading a save.
* Play the flyby before both linking sounds.
Fixes #10482, Fixes #10483.
|
|
This commit introduces the following changes:
1. Graphics::loadThumbnail()
Now returns a boolean and takes a new argument skipThumbnail which
defaults to false. In case of true, loadThumbnail() reads past the
thumbnail data in the input stream instead of actually loading the
thumbnail. This simplifies savegame handling where, up until now,
many engines always read the whole savegame metadata (including
the thumbnail) and then threw away the thumbnail when not needed
(which is in almost all cases, the most common exception being
MetaEngine::querySaveMetaInfos() which is responsible for loading
savegame metadata for displaying it in the GUI launcher.
2. readSavegameHeader()
Engines which already implement such a method (name varies) now take
a new argument skipThumbnail (default: true) which is passed
through to loadThumbnail(). This means that the default case for
readSavegameHeader() is now _not_ loading the thumbnail from a
savegame and just reading past it. In those cases, e.g.
querySaveMetaInfos(), where we actually are interested in loading
the thumbnail readSavegameHeader() needs to explicitely be called
with skipThumbnail == false.
Engines whose readSavegameHeader() (name varies) already takes an
argument loadThumbnail have been adapted to have a similar
prototype and semantics.
I.e. readSaveHeader(in, loadThumbnail, header) now is
readSaveHeader(in, header, skipThumbnail).
3. Error handling
Engines which previously did not check the return value of
readSavegameHeader() (name varies) now do so ensuring that possibly
broken savegames (be it a broken thumbnail or something else) don't
make it into the GUI launcher list in the first place.
|
|
In older releases of Myst the flyby video had the play on card change
flag set to true, causing the bug. As a workaround patch the resource to
mach newer releases.
Fixes #10472.
|
|
To make static analysis tools quiet.
|
|
Used when turning pages in the library. Some versions of the game appear
to use them.
Fixes #10474.
|
|
|
|
|
|
|
|
|
|
Otherwise the movie position is zero which leads the comparison with the
stored opcode position to always be false.
Fixes #10426, a regression from cf1171d4d9b43eba.
|
|
When videos are skipped, they would sometimes not quite be seeked to the
last frame. In that case, when they are played again without being
restarted, they would end immediatly.
Now videos are automatically rewinded when they end or are skipped so
that they always start from the beginning when played again.
Fixes #10389.
|
|
Those shortcuts are documented in the original game manual.
|
|
This fixes keyboard book page turning being too fast when holding keys
|
|
Fixes #10075
|
|
Fix undefined behaviour in variadic functions
|
|
|
|
|
|
This flag is removed for a few reasons:
* Engines universally set this flag to true for widths > 320,
which made it redundant everywhere;
* This flag functioned primarily as a "force 1x scaler" flag,
since its behaviour was almost completely undocumented and users
would need to figure out that they'd need an explicit non-default
scaler set to get a scaler to operate at widths > 320;
* (Most importantly) engines should not be in the business of
deciding how the backend may choose to render its virtual screen.
The choice of rendering behaviour belongs to the user, and the
backend, in that order.
A nearby future commit restores the default1x scaler behaviour in
the SDL backend code for the moment, but in the future it is my
hope that there will be a better configuration UI to allow users
to specify how they want scaling to work for high resolutions.
|
|
Passing a type that undergoes default argument promotion as last
argument of a variadic function results in undefined behaviour.
|
|
Fixes the per frame scripts being leaked.
Fixes #10215.
|
|
Fixes #10216.
|
|
|
|
|
|
Fixes #10114.
|
|
Thanks to dafioram for providing the replacement image.
Fixes #10115.
|
|
|
|
Fixes #10118.
|
|
It was due to inaccurate rotation movie bounds.
Fixes #10086.
|
