590829 - Avoid unnecessary seek in nestegg_track_seek

Status: RESOLVED INACTIVE

(Core :: Audio/Video: Playback, defect)

Description

[Matthew Gregan [:kinetik]]

When nestegg_track_seek loads the Cues section, it seeks to the location of the Cues, parses them, then seeks back to the parser's old location.  It then uses the newly loaded Cues to seek to the requested point in time.

There's no need to seek back the the saved parser location before seeking to the new point as it does not read from the stream between these two seeks.

ctx_restore is responsibly for restoring the old seek point and is called after loading the Cues.  This can be changed to only restore the old state when returning early due to an error.  When seeking to the new location, the ctx_restore can be avoided.

Comment 1

[Nils Ohlmeier [:drno]]

Mass closing do to inactivity.
Feel free to re-open if still needed.