Re: [xine-devel] Track name support in the demuxer plugins

[Mathieu O. <mol...@us...>]

Here's the latest versions of my patches to add support for track names
in Xine-lib and Xine-UI. The first demuxers to actually support this
feature are the AVI demuxer (audio tracks) and the Matroska demuxer
(audio and subtitle tracks).

Obviously, comments are very welcome. In the absence of any concern
about those patches, I'll probably commit them to the repository in a
few days.

Mathieu Olivier wrote:
> The attached patches add support for track names in Xine-lib and
> Xine-UI, mainly for the matroska demuxer but maybe other demuxer plugin=
s
> could use that feature, I don't know.
>=20
> My goal was to have Xine-UI displaying "en", "fr" and "Director's
> Commentary (en)" as the available audio and subtitle channels in my MKV
> test video instead of just "en", "fr" and "en" as it does currently.
>=20
> The lib patch adds a couple of functions to the API
> (xine_get_audio_name() and  xine_get_spu_name()), and the various
> constant values to support them internally (DEMUX_CAP_AUDIONAME,
> DEMUX_CAP_SPUNAME, DEMUX_OPTIONAL_DATA_AUDIONAME and
> DEMUX_OPTIONAL_DATA_SPUNAME).
>=20
> What do you think about this functionality? This patch still needs some
> code cleaning (as you'll see, logging is still activated in the
> mastroska demuxer), but since I'm modifying the API of Xine-lib with it=
,
> I would like to gather some comments early.

--=20
Mathieu Olivier  <mol...@us...>

Perfection is achieved not when there is nothing more to add, but when
there is nothing left to take away.
    Antoine de Saint-Exup=E9ry