player.c was starting getting huge (~4000 lines), and therefore harder to
navigate into.

To vlc_player_osd_Message.

No functional changes.

cf. libsmb2 upstream.

This allows layout switches between grid and list view

When the input is stopped by the user, there are lot of chances that this
access is waiting for the read command completion (smb2_read_async()).

If the read command is not fully processed (since interrupted by VLC), any
future commands will fail, like smb2_close_async() when the access is finally
closed.

To fix this issue, we switch back to the posix poll() when interrupted to let a
chance to finish the current command in order to be able to close the smb2
session nicely.

If two threads call UpnpFinish at the same time (or more precisely, if a
2nd thread calls UpnpFinish before the first one sets UpnpSdkInit to 0)
we can end up double releasing most libupnp resources

Romain Vimont authored 5 years ago and Thomas Guillem committed 5 years ago

In ThreadDisplayPicture(), when "refresh" was true, the output parameter
deadline was not written and the function returned a non-zero value.

As a consequence, in video_output.c:Thread(), the next loop iteration
waited for the max deadline (100ms). When the following frame target
date was before this deadline, the video was stuttering.

To avoid the problem, write the deadline before returning from
ThreadDisplayPicture(), so that Thread() does not wait more than
expected.

Since an existing frame is refreshed only every 80ms
(VOUT_REDISPLAY_DELAY), this happened only on low framerate videos
(<12.5 fps). Otherwise, "refresh" was always false and the problem never
occurred.

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

From the es_out, times are a triplet: pos, time, length. So they should be
sent via a single event and atomically.

This also suppress one player lock when times are updated.

cf. https://github.com/sahlberg/libsmb2/pull/94

Group every struct/enum/functions by group instead of having all structs, all
enums then all functions.

Split the header by the following doxygen groups:
 - Player instance
 - Playback control
 - Title and chapter control
 - Program control
 - Tracks control
 - Tracks synchronisation (delay)
 - Teletext control
 - External renderer control
 - Audio output control
 - Video output control
 - Player events

No functional changes.

Michael Young authored 5 years ago and Thomas Guillem committed 5 years ago

Since the underlying library supports them (since 3.0.0).

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

That support port specifications.

That was added after the 3.0.0 release.

Felix Paul Kühne authored 5 years ago and Thomas Guillem committed 5 years ago

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

The medialibrary now handles various mrls for the same device, so we can
remove this logic as it doesn't belong here anymore.

We don't need it anymore.