[MPlayer-dev-eng] [PATCH] Optional parameter for switch_audio

kiriuja mplayer-patches at en-directo.net
Fri Apr 15 04:44:49 CEST 2005


On Thu 14 Apr 2005 07:41, Michael Behrisch wrote:
> I did not compile yet but have already a minor remark.
> Please do not make demuxer_switch_audio void.
> I thought returning the number of the audio track selected would
> make it easier to add OSD support as with the subtitles

I made that change and several others. Updated patch attached.

But unfortunately it looks like the whole switch_audio thing
is going to be useful only for some audio formats, because
libfaad for example chokes on the switch and outputs garbage
for up to a couple of seconds.

> Sorry the commit for mkv came too early ;-) and I don't think I will 
> add support for other containers.

Not even AVI? There are plenty of dual AVIs out there.
Oggs too, but demux_ogg is a pretty bad mess, so it's gonna
be tough.

--
kiriuja
-------------- next part --------------
A non-text attachment was scrubbed...
Name: switch_audio_param.diff
Type: text/x-diff
Size: 8796 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050414/08793ede/attachment.diff>


More information about the MPlayer-dev-eng mailing list