[MPlayer-dev-eng] [PATCH] dvdnav - partial REVIEW

[elupus]

> I think from these the libdvdnav can give 2:
> language code = dvdnav_audio_stream_to_lang()
> physical stream id = dvdnav_get_audio_logical_stream()
> 

Yes, agreed.. But that is kinda a bug in libdvdnav, it doesn't fill the
logical stream
number in the audio / spu change event. However I still don't understand why
you need
type or number of channels. When the audio stream change even occurs, you
need to force
mplayer to player a specific physical mpeg stream. The physical number given
matches 
one to one to the lower 3 bits (mask of 0x07) of the mpeg substream id that
should
be available in mplayer.

So the only function you need (if you are not using the events) would then
be
like you say dvdnav_get_audio_logical_stream() (wich is missnamed as it
actually
returns the physical stream number). 

/Joakim