(And remove an useless constant function pointer.)

The function pointer (in vlc_gc_*) was useless here.

(This saves one function pointer.)

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Use /org/mpris/MediaPlayer2/TrackList/Append instead of / as the
AfterTrack argument to org.mpris.MediaPlayer2.AddTrack for appending

The single instance mode currently tries to contact another VLC instance
through dbus in order to find out if one is running. With dbus activation,
it results to the dbus daemon launching a new VLC instance while the
first one blocks until the new one answers the call

This patch uses org.freedesktop.DBus.NameHasOwner instead in order to get
an answer from the dbus daemon without it launching a second instance.

 * use a thread instead of short interval polling,
 * do not assume latency is shorter than a period,
 * negotiate sample format instead automatically,
 * map all known ALSA formats to VLC codecs,
 * negotiate channels and sample rate,
 * partial support for 3-6 channels (missing preferences),
 * negotiate buffer and period times according to caching,
 * tolerate off-by-one period size due to rounding,
 * use the "default" device by default rather than "hw",
 * remove old-style hardware-only suggestions in debug messages.

This should fix support for or work around bugs in, some drivers:
 * explicitly disable soft-resampler,
 * define access mode before sample format,
 * set buffer before period,
 * use period time rather than period count.

modules/stream_filter/dash/http/HTTPConnection.cpp was case-sensitive
when checking Content-Length.  RFC 2616 section 4.2 states "Field names
are case-insensitive"

The http server built in to the JDK 1.6 uses a field name of
"Content-length" which reveals an incompatibility of VLC's DASH module.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Change the shortcut for convert/save to 'Alt+O' as 'Alt+C'
conflicts with the Cancel button.
Signed-off-by: Vikram Narayanan <vikram186@gmail.com>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

If the decoder returns a new size via the crop rect, but hasn't
explicitly indicated a plane height (via the nSliceHeight parameter),
keep the frame height as plane height instead.

Also do the same check for stride vs width. The decoders that had
problems that the nSliceHeight fixes do set nStride properly, so
that part won't be needed there, but might be needed on some
platform.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Pointed-By: Clément Lecigne

It is an OpenDML extension that contains the aspect ratio (it closes #6777).

This way, the official atomic functions will be used when available,
rather than VLC's own implementation.

This commit also enables proper atomic operations on some platforms
or with some toolchains. __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 is not
defined on some platforms such as ARM Linux, nor by other compilers
than GCC such as LLVM/clang. Then atomic operations wrongly fell
back to the lame mutex-based implementation.

This will break support for some old compilers and old or irrelevant
instruction set architectures. They would need to provide replacement
for the Intel-originated __sync_* builtin functions.

Since this constitutes mostly of macros and type definitions, it would
not fit too well in compat/. Most of the code would end up in
<vlc_fixups.h> rather than compat/. Moreover, I doubt that those
functions would be detected properly with AC_CHECK_FUNCS or
AC_REPLACE_FUNCS.

Anyway, VLC already has a separate header, and it will need to keep it
until <stdatomic.h> can be relied upon... many years from now.

This is now the same as was already implemented in the
set_relative_playlist_position_and_play() method.

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

macosx: improve the handling of broken prefs and make sure that the user is unable to re-break them without using the command-line or a text editor

This reverts commit 8bc285bd.

The original purpose for this code is fixed somewhere else today.

close #6769

In particular, V4L is (usually) an access_demux rather than an access.

Format the text. If it is identical and all parameters are unchanged,
there is no need to update - even if the format contains a $ sign.