mediacodec: factorize release_output_buffer

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

mediacodec: handle error_state in one place

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

mediacodec: process input buffers in only one place

(behavior not changed)

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

demux: avi: improve broken index offset heuristic (fix #14120)

Avoids using movi position as offset when the broken index refers
to beginning of file and the first sample size is exactly movi's
offset.

udp: fix locking (fixes #14234) and cancellation

enable the macosx GUI to handle negative stop-time

Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

direct3d11: implement the pixel format fallback

NV12 is not supported in Windows 7 [1] so we need to fallback to RGBA

1: https://msdn.microsoft.com/en-us/library/windows/desktop/bb173059%28v=vs.85%29.aspx#DXGI_FORMAT_NV12

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

direct3d11: don't crash when exiting with an error

It happens when opening the D3D11 device fails

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

direct3d11: code cleaning and typos

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

direct3d11: D3D11_CREATE_DEVICE_DEBUG requires VisualStudio or Windows SDK

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

demux: ts: only interpolate PCR on missing PCR

Because if the PCR exists and has an offset,
the first PCR update will be interpolated
and no following pcr/dts offset fixup will
work without delaying playback by the
real - interpolated pcr.

demux: ts: correctly update current chain when dequeuing

Upnp: remove trailing space

LUA is still GPLv2+

Remove bonjour in access_output

It's a great idea, but not the best implementation and broken since a
long time.

direct3d11: give enough room for the \0 in the string

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

Qt is GPLv2+

vpx: fix leak

RSS: remove no-op

"end" label is only reached if psz_eltname is NULL.

shine: avoid double free

If subsequent malloc or realloc failed, free would be called on a dangling
pointer.

access: archive: remove dead initialization

block_helper: remove dead assignments

macosx: video effects: fix threshold checkbox visibility

decoder: fix race / incorrect assertion

wasapi: add loopback mode

wasapi: audio capture client module (fixes #7205)

puzzle: fix error check, do not leak pictures on error

close #14006

Real RTSP: discard unsupported legacy username and password syntax

This is better than leaking them to the DNS resolver. This also avoids
confusing error messages if live555 is missing or failing.

mux: mp4: handle box/bo failed alloc/realloc

refs #14193

mux: avi: handle failed reallocs

refs #14193

mux: avi: don't try to delete failed stream

mux: avi: fix leak on format failure

vlc_bits: use vlc_common and guards against failed realloc

refs #14193

decoder: reduce lock scope and cosmetic

p_owner->b_paused can only be written by the owner, so the owner thread
does not need to lock to read it. (However, locking is necessary to
write it from the owner thread, or to read it from the decoder thread.)

aout: remove aout_DecIsEmpty()

decoder: drain the audio output properly

aout: add wait parameter to aout_DecFlush()

decoder: cosmetic changes

decoder: inline DecoderSignalWait()

decoder: do not mix and match condidition variable and mutex pairings

decoder: do not wait for buffering when there is no data

This removes small hacks surrounding this corner case.

decoder: remove BLOCK_FLAG_CORE_EOS

Do not rely on allocating a block for draining the decoder and output.

fifo: fix FIFO bytes size computation

decoder: remove unnecessary special case

All audio decoders support the *pp_block == NULL case, and return NULL.
This enables audio decoders to drain cleanly.

decoder: simplify audio path

DecoderPlayAudio() has no effects with a NULL block.

macosx: fix main menu initialization order on startup

MainMenus p_intf can be already accessed before
applicationWillFinishLaunching was called.

This should fix quite some crashes reported by the users.

es_out: use input_DecoderDrain()

decoder: add input_DecoderDrain()

macosx: fix wrong ref counting in media info

decoder: factor some common code into DecoderUpdateFormatLocked()

decoder: reorder to avoid forward declation, no functional changes

decoder: fix data race in sout

decoder: merge b_exit into b_flushing

There is nothing special about exiting at this level. The decoder
should discard its buffers and return to reading the input FIFO at the
earliest (where it gets canceled). This is the same as flushing.

decoder: fix data race in input_DecoderChangePause()

See also 0fde3beaa85528e555bac9c1b327ba5a32da67e9.

decoder: fix data race in input_DecoderFrameNext()

See also 0fde3beaa85528e555bac9c1b327ba5a32da67e9.

es_out: do not start buffering when already buferring

This fixes an assertion failure.
Regression from 621400186a49b282197267e588a030b425829a64.

decoder: add input_DecoderFlush()

macosx: Update progress dialog on the main thread, make check thread safe

This should fix some crashes as reported by the users.

vlc_plugin: fix non-LGPL plugins meta infos

vlc_plugin: allow overriding module meta-infos

directsound: fix TimeGet returning a positive value on error

Fixes #14186

a positive HRESULT means it succeeded.

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

Added forward and rewind (30s) buttons to mobile HTTP remote control

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

decoder: fix race in spu_new_buffer

There is a race when starting video with subtitles at a given position (via
--start-time).

If all decoders are flushed early, the video decoder may not have created the
Vout via the decoder_NewPicture call. In that case, spu_new_buffer will be
blocking for 6 seconds. Indeed, spu_new_buffer can block for maximum 6 seconds
when it's waiting for a Vout.

To solve this race, abort spu_new_buffer if decoder is flushing.

How to reproduce the race condition:
./vlc --start-time 3600 <video_with_subtitles>

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

Update bookmark list before showing dialog.

Fixes #2100.

Override toggleVisible of BookmarkDialog to call update before using toggleVisible of QVLCFrame

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

libass: use the visible area dimensions, not the input dimensions

Fixes incorrect offset when playing http://streams.videolan.org/misc/test-files-ccc/vsfilter_general_TestCase.mkv

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

fix teletext framing code in DVB PES packets ignored

Check the teletext framing code in PES buffer for each of the received lines
and only copy those with correct framing code (p_block->p_buffer[3]) to the
p_sliced buffer that is later forwarded to ZVBI vbi_decode. Invalid lines
will not reach vbi_decode anymore.

When also packets with erroneous framing code are sent to vbi_decode (often
0x00 with some noise), in most cases those are decoded as packet 1/2,
causing the second text line on the teletext page to be overwritten with
spaces. So we need to avoid sending such packets with invalid framing code
to vbi_decode.

Close #14191

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

Win32: don't call setsockopt if compiling for Windows 8.0

udp: adjust receive buffer for windows 7 and earlier

Try to detect runtime what windows version it is running to see if
receive buffers should be increased or leave to OS handling.
Fixes #14200

ALSA: request large enough buffers first (refs #10422)

With the current input clock and decoder buffering code, getting large
buffers is essential. So favor that over having short audio periods.
(Ideally the audio period should probably be an half or a third of the
PTS delay, rather than hard-coded to 40 ms, but that is a separate
issue).

vlc_bits.h: drop len field

mux: mp4: vlc_bits regression (fix #14192)

commit 6c9bc13fca989feca53a1ccdd609d6c6995a3879

playlist: remove activity callback in favor of input-current

playlist: move input-current, fix race

See also a6b259b733a0ba0c064377cf07b5bd0c33584b17.

block: remove block_FifoPace()

decoder: avoid block_FifoPace(), fix races

decoder: remove unused parameter

DecoderSignalWait() had no observable effects if has_data was false. It
could wake up the condition variable, but no thread would wait on that
event.

block: remove block_FifoWake()

decoder: avoid block_FifoWake()

udp: use the new FIFO functions

block: rewrite block_Fifo(Empty,Get,Put) with the new functions

block: add low-level functions for block FIFOs

In some cases, the thread(s) consuming a FIFO needs to wake up in other
circumstances than the FIFO being non-empty. For that purpose, this new
set of functions is vastly more flexible than block_FifoWake().

block: split block_t and block_fifo_t code

decoder: remove ARM-specific limitation

There are ARM servers and or HD/UHD-capable STBs. The distinction no
longer makes sense.

es_out: remove tautology

es->p_dec is obviously non-NULL when the loop starts, and the loop
cannot change the value.

decoder: remove tautology, add asserts to waiting code

decoder: remove invalid call, fix races

Calling input_DecoderStopWait() from the decoder thread makes no sense
as it signals a condition variable that only the decoder thread itself
would ever wait on.

More importantly, the call breaks assumptions made by the input thread.
For instance, input_DecoderDecode() assumes p_owner->b_waiting is only
changed by the decoder owner (i.e. input) thread.

vlc_bits: add bo_set_64_be

vlc_bits: fix bo_set_32be

demux: ts: fix IOD signedness/warning

demux: ts: fix opus signedness/warning

opus: fix duration signedness

demux: ts: fix signedness/warnings

MKV: factorize the es_out_Send() and es_out_Control() calls

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

DirectX/GDI: use COINIT_MULTITHREADED instead of COINIT_APARTMENTTHREADED

The renderer may be updated in the DMO thread which uses COINIT_MULTITHREADED

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

MKV: packetized codec can use VLC_TS_INVALID as i_pts for extra Block frames

Fixes WMA 9.1 Lossless playback through avcodec

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

MKV: fix COOK/ATRAC3 seeking

Fixes #14210

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

MKV: fix ATRAC3 playback

Fixes #14205

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

MKV: A_MS/ACM is not packetized

Just like in AVI
See modules/demux/avi/avi.c l461

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

DMO: use COINIT_MULTITHREADED instead of COINIT_APARTMENTTHREADED

Fixes #14202

All the other CoInitializeEx() calls and now using multithreading, except for
this code which is called in UpdateRects()
See modules/video_output/msw/common.c l286

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

mux: mp4: handle both short and long NALU startcodes (fix #14185)