[MPlayer-dev-eng] [PATCH] Add support for default of multiple video tracks

[Ingo Brückl]

I wrote on Thu, 30 Jan 2014 02:05:13 +0100:

> Reimar Döffinger wrote on Wed, 29 Jan 2014 21:36:30 +0100:

>> You put it in the code handling program switching.

> Although I don't fully understand this part of mencoder, I see now.

>> I meant putting it right above the select_audio just like in mplayer.
>> Though double-checking the interaction with this program handling might
>> be necessary.

> I am just asking myself whether the demuxer_switch_video() call would be
> redundant now and can be removed.

> If prog.vid can be -1 and if demuxer_switch_video(..., -1) actually
> performs something, the call must remain. Else it will be superfluous,
> because the new select_video() will do the job now.

After a closer look into demux_ts.c I'd say prog.vid can't be -1 (only -2 if
there is no video), so the demuxer_switch_video() call can be removed as seen
in the patch I've suggested.

After several smoothly tests I will apply now.

Ingo