test: check full help

This can often catch bugs in the plugin descriptors and modules core.

modules: restore empty strings in cached choices list

audioqueue: don't use the main CFRunLoop for the (empty) data callback, but one of AudioQueue's internal threads

Otherwise, we will encounter delays of up to 10 secs when stopping the AudioQueue

audioqueue: fixed incorrect buffer allocation which prevented playback of certain samples

ios: don't compile with --enable-debug with -g only by default

timecode: generate a clock/timecode as a subtitle track

This is mainly intended to be used as a slave input, e.g.:
 # vlc video.mkv --input-slave timecode:// --timecode-fps 30/1001

cache: store whether there is a choices callback or not

ConfigLoadString loses the last character of strings

ConfigLoadString allocates memory for 'size' bytes and copys a NULL terminated
string to this buffer.
ConfigSaveString instead expects the buffer to be NULL terminated but:
 - the 'size' saved to the cache is the char count
 - the string is written without terminating NULL.

So obviously ConfigLoadString does not match the behaviour of ConfigSaveString.
It seems to me that the cached module configuration is not working at all due
to comparing the module file names within CacheFind - this should never have
found a match.

I think this would need some more testing.

Regards,
Mario

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

Qt: PLSelector: add side icons

DEMUX_SET_NEXT_DEMUX_TIME: fix documentation

Qt: PLSelector: add side icons

Qt: PLSelector: Pack entries to the left

audioqueue: fix compilation warnings

asf demux: rewrite DemuxPacket

Split it in several functions

modules/services_discovery/sap.c: avoid out-of-bounds write

After OpenDemux reads data using stream_Read(), it writes a '\0' to
the buffer after the newly-read data, but if the stream returned exactly
i_read_max bytes, this '\0' will end up just past the end of the allocated
psz_sdp array (see the call to realloc at the beginning of the loop).
Adjust the realloc call to allocate this one extra byte.

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

asf demux: cosmetics

merge declaration and initial assignement

ASF: don't print debug infos in releases

asf demux: fix #8024

Replace macro with static inline and use bounds checking

PulseAudio: implement mute/unmute while not playing

PulseAudio: implement and fix changing device while not playing

PulseAudio: implement sinks list (i.e. output devices)

PulseAudio: revector, no functional changes

PulseAudio: report current output device

PulseAudio: implement output device change

aout: remove has-choices flag on "device", not applicable anymore

Avoid zero-length array

While useful, this is not permitted by the standard specification.
This patch will waste a little bit of memory in affected allocations...

minimal_macosx: remove unused variable

avcodec: fix warnings

Lua RC: comment out broken adev implementation

This reverts commit 5086a9ed9e848a54f02f585e80c9f2730868bca9.

Old RC: use audio output device functions and clean up

Fix corrupted rendering of EIA-608 captions

When we moved the code to use the legacy method *or* the new method,
we lost the functionality that actually initialized the memory allocated
for the entire subpicture region (including the part where captions
weren't being rendered.

This was most obvious when regression testing on OSX, where it
resulted in old captions continuing to appear on screen even
after being cleared, or garbage in the render buffer.

Add a line that effectively zeros out the buffer prior to use.

Signed-off-by: Rafaël Carré <funman@videolan.org>

Win32: we need gai_strerrorA and not gai_strerrorW

Atmo: use LoadLibrary instead of LoadLibraryA for a fullpath DLL

Win32: undef lseek before redefining it

Win32: use _snwprintf instead of swprintf

swprintf is not the standard function you are looking for

WinVLC: use _snwprintf that is conforming to the standard

Skins2: try to fix compilation in ANSI mode

Skins2: reimplement rmDir on Windows with ShellApi

DirectDraw: fix memleaks introduced by 46b26be57

DirectX: use DirectDrawEnumerateExA/W when GetProcAddress

Atmo: fix memleak introduced by 59f746f6

Win32: better eject code for UNICODE-APIs

D3D; fix warnings

Remove untested strncasecmp replacement

Since it never built, it was never needed on any platform

contrib: ffmpeg: account for libvpx decoders name change

'libvpx' decoder doesn't exist anymore but we continue to disable it anyway
for people building and older version

Win32: kill DirectX warnings

DirectDraw: port to UNICODE friendly APIs

This would need more testing and close look

Atmo: port Win32 code to UNICODE friendly APIs

This needs testing

Freetype: use ANSI version of registry open functions

Skins2: port Win32 code to UNICODE

This would deserve a second look, though

Qt4: force registry functions to ANSI mode

Win32: fix eject behaviour with UNICODE

NTservice: force use of ANSI versions of APIs

aout: remove choices list on the device variable

This does not support hot plug and unplug of devices.

Also, it prevented selecting a device not in the choices list due to
checks in the variables subsystem. (At least with ALSA, not all valid
devices are included in the suggestions.)

hotkeys: use aout_DeviceGet() and aout_DevicesList()

aout: remove temporary hack on the "device" variable

aout: add some debug when restarting

Far too many bugs in this code path recently.

aout: fix crash when changing device fails

hotkeys: use aout_DeviceSet() directly

Using var_Set("device") is wrong: the "device" variable holds the
currently used output device (as set by the aout plugin), not the
intended output device (as requested through the user interface).

Also fix a memory leak.

Direct2D: fix GetProcAddress call for UNICODE

Windrive: use TEXT macro as applicable

Win32 eject: use correct type for MCI_OPEN_PARMS structure

Qt: use TEXT macro with RegisterWindowMessage

DxVA2: fix GetProcAddress usage under UNICODE

Win32: Fix DMO compilation behaviour with UNICODE

CrystalHD: fix compilation behaviour with UNICODE

DirectX: fix GetProcAddress call with Unicode

CDROM: use CreateFileA for opening drive letters

If I understood correctly, drive letters are always ANSI

Win32 screen: use TEXT macro

Win32: fix GetProcAddress call with UNICODE

Weirdly, there is no GetProcAddressW, indeed

Win32: use OutputDebugStringA directly

Since OutputDebugStringW is just a wrapper around OutputDebugStringA

Win32: use TEXT macro for SetConsoleTitle

Ogg: display debug about Annodex info

Ogg: factorize reset function

Ogg: implement SET_SEEKPOINT

Close #6895

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

Ogg: remove unused function and variable

Ogg: use b_force_backup as a boolean

Ogg: clean header and remove unused members

Ogg: use calloc

vdpau: use ctz()

vlc_common: add ctz() to count trailing zeroes

Fix compilation without Xlib

News

vdpau: VDPAU hardware decoding with avcodec

This requires bleeding-edge libavcodec.

XCB/screen: use shared memory to reduce overhead (if available)

Add block_shm_Alloc()

macosx: fix and improve window level handling

    - fix behavior of video-on-top by adapting to vout windows handling changes
    - set all windows to status level if one vout window has this level:
    This avoids that video effects panel, audio efffects panel etc. pp. are opened behind
    a vout window. Now they can be used as usual.

    Please note, that due to the type of these panels they do not remain visible
    when VLC gets inactive.

Don't call the legacy Freetype background renderer if the EIA-608 bg render feature is used.

If we're using the new background renderer which just renders the
background under the text (instead of the entire subpicture block),
then don't call the legacy render as well.  This prevents a alpha-
transparent box from being shown when rendering EIA-608 captions
if the user has the Freetype background opacity set to a nonzero
value.

Signed-off-by: Rafaël Carré <funman@videolan.org>

ram: fixed typo

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

Win32: set AppContainer and no-SEH flags on PE

VLC is not the place for Kassdédi
(cherry picked from commit d6699a1f4ae27f04079edef946e2bf5a7fdb446b)

Conflicts:
po/fr.po

Avoid inifite loops and stack explosion when parsing broken files and ignore lvl 1 unknown elements.

Fix #8013

avcodec audio decoder: remove SplitBuffer()

That function was introduced years ago to account for bugs in aout,
which do not exist anymore.

Incidentally fix WMA.

avcodec audio decoder: use VLC_TS_INVALID

avcodec audio: factorize error case

Add support for rendering EIA-608 captions with a black background

Place a black background behind caption text.  We cannot use the
existing "background" feature for subdecoders because it fills the
entire subpicture block, and in the case of the EIA-608 decoder
would result in a large square (since we send all lines to
the subpicture decoder even if they are blank).  This approach
allows for rendering of the background even in such cases, while
preserving proper layout of the captions.

Signed-off-by: Rafaël Carré <funman@videolan.org>

avcodec: allow hardware acceleration plugin to use pic->opaque

contrib: put hash in ffmpeg tarball name

omxil: enable support of mp3, vorbis and wma

Update audio_format_table to allow selection of OMX
components which support mp3, vorbis or wma role.

Signed-off-by: Rafaël Carré <funman@videolan.org>

omxil: Enable pts mode for OMX.STM.Video.Decoder

OMX.STM.Video.Decoder requires input buffers with pts timestamp.
A debug trace is added to inform if pts mode is enabled and for
which codec.

Signed-off-by: Rafaël Carré <funman@videolan.org>