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>

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

A switch-case that introduces variables should be scoped, as it
otherwise is ill-formed to add another case after it.

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

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

As KaxSimpleBlock and KaxBlock inherit from the same base we can get
rid of all these nasty if-else's by creating an object that refers to
the one we are interested in.

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

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

VLC_DEMUXER_EOF is not correct if we abort due to an error.

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

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

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

This deletes previous output if the section is cancelled.

refs #20835

Overly longs filenames are now handled correctly.

refs #20835

There does not appear to be any real user. The "broadcast"
functionality can be reproduced more simply with the LibVLC media
player API. The "VoD" functionality does not make much sense outside of
the VLC executable process.