Previously, the file name label was set before the window was actually loaded so this silently failed as the object was nil. This ensures the label to be set once the window was loaded when resuming a media item for the first time during the lifetime of the app.

Mikhail Gusarov authored 5 years ago and Felix Paul Kühne committed 5 years ago

Existing logic of storing playback position for macOS excludes positions within
first/last 5%.

This heuristic is fine for short videos, but for a longer one, say a 6-hour one,
it means the position is not stored for the first/last 18 minutes, which is a
nuisance.

This change adds an upper limit for the size of excluded positions.

Signed-off-by: Felix Paul Kühne <felix@feepk.net>

Mikhail Gusarov authored 5 years ago and Felix Paul Kühne committed 5 years ago

This refactoring is a prerequisite for the next commit that improves the
decision for long video files.

Signed-off-by: Felix Paul Kühne <felix@feepk.net>

Just because the generation was started, does not mean it succeeded, so we cannot trust the state but need to check for the URL's validity.

Alexandre Janniaux authored 5 years ago and Thomas Guillem committed 5 years ago

Silence -Wstringop-truncation warning.

Signed-off-by: Thomas Guillem <thomas@gllm.fr>

Regression from d23b4527

refs #22524

Refs #22524

As we will save the playback states in the media library from the
player, we need to ensure the media library is still available when
doing so.
Since the last save can occur while destroying the player and it's
associated input, we need the media library to be destroyed after the
player, and therefor the playlist.

The input already takes care of it by probing the demux in
UpdateTitleSeekpointFromDemux

Alexandre Janniaux authored 5 years ago and Thomas Guillem committed 5 years ago

vlc.player.add_subtitle accepts a second parameter autoselect which
default to fals.

Signed-off-by: Thomas Guillem <thomas@gllm.fr>