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

demux: ts: update pid states before parsing data

macosx: fix video-wallpaper mode by ignoring core events

wallpaper mode is more complicated than setting fullscreen and
level values. Thus, it is implemented in the macosx UI, and we
need to ignore some core events for proper functioning.

close #14151

Map Canopus HQX decoder

Add Canopus HQX FourCC

Map TDSC avcodec

Add TDSC fourCC

Update NEWS

MKV: avoid using the last_dts of a track we know is not used anymore

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

mediacodec: don't use MTK MPEG4 decoders

Fix a crash with android mediaserver

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

Win32: fix uninitialised variable and double free in drive letters mode

Close #14140

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

Consistent strings to avoid duplications

sftp: add options for username and password

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

MKV: fix compilation

MKV: remove unused i_time_offset variable when seeking

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

MKV: no need to add VLC_TS_0 here, it's already shifted

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

MKV: init the index time to -1 as it was intended

As witnessed here git.videolan.org/?p=vlc.git;a=blob;f=modules/demux/mkv/matroska_segment.cpp;hb=HEAD#l1427

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

MKV: divide i_default_duration by 1000 once and for all

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

wpl: Remove debugging leftover

playlist: Rewrite WPL module

It is now using our XML parser, and is licensed as LGPL

Remove WPL module

sftp: do not try to open directory as a file

Usually opening a directory as a file succeeds.

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

mediacodec: check getOutputBuffers and getInputBuffers

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

mediacodec: factorize exception check

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