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)

mux: mp4: invalid cast

commit 8fd834512cf778a89d0fbf2540b0055dec002f42.

httplive: leaks on playlist reload/merge (fix #14171)

demux: ts: gather data for uncreated es

should go to pre es queue otherwise first frames/gop are dropped

Interface: HTTP server: set proper align when expanding menu buttons

Fixes #13855

The small menu buttons on the left doesn't move downwards anymore when
the menu is expanded

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

sftp: implement directory listing

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

MKV: use the more accurate i_pcr for GET_TIME and GET_POSITION

similar to what avformat does

I forgot to re-submit that one after the other commits went through.

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

block: simplify block_FifoRelease

Since the FIFO is being destroyed, there is no need to lock and to
update the FIFO internal state.

udp: avoid alloc/free on I/O error

decoder: fix data race in input_DecoderIsEmpty()

p_dec->fmt_out is owned by the decoder plugin, and can only safely be
accessed by the core from within decoder callbacks, notably the format
update callbacks.

Outside that context, p_owner->fmt has to be used. It contains a copy
of p_dec->fmt_out at the last format update.

decoder: remove redundant call to block_FifoEmpty()

block_FifoRelease() takes care of it.

Revert "MKV: Always set an i_pts in a lace otherwise it disturbs seeking performance"

This reverts commit 6a16055a3331a4022d12ba29ab002a3257e4c89c.

Fixes #14194
Vorbis doesn't work properly with forced bogus PTS.

The seek issue in #13575 is solved with the DirectSound fixes from #14186

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

do not install dsm plugin twice

On ma, 2015-03-16 at 13:59 +0300, Rémi Denis-Courmont wrote:
> Le 2015-03-16 13:24, Petri Hintukainen a écrit :
> > I tried to enable dsm access plugin in Linux (to get directory
> > browsing
> > support). But "make install" failed with error:
> >
> > libtool: install: error: cannot install `libdsm_plugin.la' to a
> > directory not ending in /usr/lib/vlc/plugins/services_discovery
> >
> > It looks like the module is installed twice: first to access/, then
> > to
> > services_discovery/. Installing it only to services_discovery/ seems
> > to
> > work.
>
> The (rest of the) Makefile rules are in access/ though...

Removing it from services_discovery/Makefile.am in commit
http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=42350148049e6a91db1f6876d1e0765640da0f7d
didn't fix the problem with make install.

With
../configure --prefix=/usr --enable-dsm --disable-smb
make && sudo make install

I still get

libtool: install: error: cannot install `libdsm_plugin.la' to a
directory not ending in /usr/lib/vlc/plugins/services_discovery

Alternative patch attached.

(or maybe I have broken libtool ... ?)

>From 43e5502bb44f8611eb1d61cc8fa6c1c711dce32b Mon Sep 17 00:00:00 2001
From: Petri Hintukainen <phintuka@gmail.com>
Date: Tue, 17 Mar 2015 09:52:47 +0200
Subject: [PATCH] dsm: fix runtime library search path

Fixes "libtool: install: error: cannot install `libdsm_plugin.la'
to a directory not ending in /usr/lib/vlc/plugins/services_discovery"

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

MKV: fix a memory leak

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

Qt: do not emit signals from the MIM constructor

Should close #14162

Qt: MIM: code cosmetics

Qt: MIM initialize p_input through constructor parameter

mux: mp4: fix NALU size endianness

vlc_block: fix NULL dereference in block_ChainGather

demux: mp4: don't NULL dereference on failed realloc

demux: ts: set fmt id on all es and add collision avoidance for extra es

direct3d11: Fix mingw build

demux: ts: fix potential segfault

introduced by 1dec820c43290fe19ac8b291d70353943275b8fc

demux: ts: fix uninitialized fields with teletext es

demux: ts: update probing

fixes invalid length/time with arib
and handles shared pid

MFT: GUID_AC3 should be defined in Mingw-w64 4.0

Of course, it's technically a GCC bug, because selectany is broken, but
there is no simple fix for that.

ALSA: log the device string that is actually opened

ALSA: only append AES parameters to iec958 and hdmi plugs

Other ALSA plugs do not support those parameters.

Regresion from 7a19c0b41e4c43392f423338d41b0bcebc3e5329.

build: check for gmtime_r and localtime_r with the new macro.

This makes sure that the two functions are found when building on mingw32,
as the functions are defined as static inline rather than as library
symbols.

Signed-off-by: Diego Elio Pettenò <flameeyes@flameeyes.eu>
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

build: create a macro to check for, and replace, possibly-inline functions.

mingw (both 32 and 64) provides a number of functions that have no C
linkage, but are only available as static inline. Define a macro that can
check for the function declaration but acts like AC_REPLACE_FUNC.

Use the new macro for asprintf/vasprintf (previously implemented in
configure.ac directly).

Signed-off-by: Diego Elio Pettenò <flameeyes@flameeyes.eu>
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

compat: fix localtime_r() replacement

 - Do not clobber thread-specific state on Windows
 - Be thread-safe on non-Windows

win32: add assertions for unchecked CoInitializeEx() calls

dmo: fix format string

dmo: avoid useless string conversion

dmo: fix invalid string casts

dtv: fix build with Linux DVB versions <= 5.8 (fixes #14156)

ts: dvbpsi: don't spam with irrelevant errors

should be fixed from libdvbpsi >= 1.2.0

SD: do not install dsm plugin twice

MKV: Always set an i_pts in a lace otherwise it disturbs seeking performance

Using VLC_PTS_INVALID disturbs seeking.

It only applies to audio frames which are meant to be contiguous.

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

MKV: rename the Matroska timestamp fields to differentiate from VLC_TS ones

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

Win32: Fix supported types not registered properly by NSIS installer

A bad parameter definition in function RegisterExtension resulted in the same value entry being written every time ("${EXT}"="").

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

subsusf: Remove unneeded initializations

Add TTML fourcc

Add tshd and rv64 FourCC for H264

mediacodec: add a function that checks if codec is blacklisted

This also fixes a crash because of a missing __MIN(strlen()) for MTK MPEG4.

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

MKV: send the proper PCR before es_out_Send() of the packet

replaces previous similar patches, now with proper VLC_TS adjusting

Fixes #13575
Fixes #11847 which had time display issues

The new PCR logic is borrowed from
http://git.videolan.org/?p=vlc.git;a=blob;f=modules/demux/avformat/demux.c;hb=HEAD#l776

A PCR is always sent now, unless i_last_dts is not set, meaning the first
packet doesn't have a proper DTS (invalid data at the beggining)

The i_pts is not set to 0 anymore in a lace as it disturbs seeking performance

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

MKV: no need to shift the dts/pts twice in some cases anymore

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

MKV: force 64 bits constants the right way

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

MKV: fix mixup between VLC timestamps and Matroska timestamps

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

MKV: don't use i_last_dts if it's invalid

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

demux: ts: only set self pcr to its program

We can't support shared pid/pcr for now

demux: ts: check other programs/shared pcr for PCRHandle

demux: ts: reject shared pes for now

We need one ES by program

demux: ts: fix wrong flush condition

demux: ts: use a more sane default pcr pid

demux: ts: do not create multiple stream filter

can happen with multiple arib programs

demux: ts: apply pcr filter last and update it on fixup

demux: ts: validate es from non filtered streams

packetizer: mpeg4audio: don't spam with warnings

If frames have CRC, console is spammed.
CRC warning should happen only once.

demux: ts: directly ignore scrambled content

demux: ts: typo and add debug info

demux: ts: fix delayed es creation after filtering