[MPlayer-dev-eng] [PATCH] Default to the first found ES in MPEG TS

[Nico Sabbi]

Nico Sabbi ha scritto:
> Yue Shi Lai ha scritto:
>> Hi,
>>
>> I found that the logic in libmpdemux/demux_ts.c, which by default 
>> choses the _last_ ES (if no PID is requested, it just loop through 
>> and only the last PID is kept), is very annoying when playing back 
>> MPEG TS with multiple video and audio ES. Usually the last ES tends 
>> to be commentary or picture-in-picture features, and the user is 
>> forced to look through the TS structure every time to get the PID of 
>> the "normal" streams.
>>
>> I am not sure if this behavior occurs with other types of multiplexed 
>> streams, but attached is a simple 3 line patch that fixes this for 
>> MPEG TS. I would be glad for any comments.
>>
>> Best,
>>
>> Yue Shi Lai
>
> I don't see why this patch should make the stream selection more 
> deterministic: mpeg-ts is and will always be full of channels
> and even with your patch applied the chosen streams will be as random 
> as they currently are.

oops, I meant "as it currently is".