fixup of cfea200c

The second arg of ES_OUT_SET_ES_BY_ID was ignored (and was false for input
slaves).

Filip Roséen authored 6 years ago and Hugo Beauzée-Luyssen committed 6 years ago

A widget will (per default) receive focus when it is displayed,
something which is not ideal for our fullscreen controller (as it may
result in VLC grabbing focus from whatever window the user is actually
interested in at the time).

fixes #8712

Signed-off-by: Hugo Beauzée-Luyssen <hugo@beauzee.fr>

Count ESes using the foreach_es_then_es_slaves() helper and add the "Disable"
track from legacy var callbacks.

Add the EsOutGetSelectedCat() helper to replace var_GetInteger( p_input,
"*-es").

The variables "video", "audio", and "spu" were reset to true only when an ES
was changed from "*-es" variables.

To fix this issue, add a new arg to the ES_OUT_SET_ES_BY_ID control that allow
to bypass initial user preferences regarding track selection.

Move these 4 variables from input_LegacyVarInit() to input_ConfigVarInit()
Indeed, these variables are needed, as read-only, for input_thread or the
es_out initialization.

Since this module handle the same way "sub-fps" and "sub-original-fps" (and
because fetching "sub-original-fps" from the input is a HACK).

This commit merge "sub-original-fps" and "sub-fps" variables into one. The
"sub-fps" will be always valid. It will be, by priority order, the user choice,
then the master demux fps.

To a version which will link on macOS

All expected values (vlc_ml_file_type_t) are explicitely handled by the
switch

Since flexible arrays are only language extensions, it's unsafe to do
any assumption on what decltype(<flexible array>) will yield.
This fixes the build on gcc 5 (at least)

It's been unused since b49f27fc

duplicate spu issue with demuxers not flagging discontinuity

Filip Roséen authored 6 years ago and Steve Lhomme committed 6 years ago

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

Filip Roséen authored 6 years ago and Steve Lhomme committed 6 years ago

f87c4418 + a6fde345 did not do a very good job at replacing variable
names (resulting in an inability to enable MKV_DEBUG) as the variables
referenced has new names.

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

Filip Roséen authored 6 years ago and Steve Lhomme committed 6 years ago

Prevent potential null-pointer dereferences.

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

They will be enabled if the dependency is there. This fixes builds with
configure.sh & --disable-sout

Refs #20814

Filip Roséen authored 6 years ago and Steve Lhomme committed 6 years ago

fixes: #20196

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

Filip Roséen authored 6 years ago and Steve Lhomme committed 6 years ago

fixes: #14577

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>