[MPlayer-dev-eng] [PATCH] stream layer support for .ifo files

[Andrew Calkin]

2007/12/3, Vladimir Voroshilov <voroshil at gmail.com>:
> 2007/12/2, Alban Bedel <albeu at free.fr>:
> > On Sun, 2 Dec 2007 17:53:38 +0600
> > "Vladimir Voroshilov" <voroshil at gmail.com> wrote:
> >
> > > 2007/12/2, Alban Bedel <albeu at free.fr>:
> > > > On Sat, 1 Dec 2007 17:57:27 +0600
> > > > "Vladimir Voroshilov" <voroshil at gmail.com> wrote:
> > > >
> > > > > "mplayer -dvd-device /path/to/my/dvd/on/hdd dvd://"
> > > > > AFAIK, it is not possible to make latter call from menu.conf.
> > > >
> > > > Just extand the dvd stream to be able to take the device from the
> > > > path part of the url, then dvd://1//path/to/my/dvd/on/hdd will do
> > > > what you want.
> > > >
> > >
> > > Such support is simple to implement, but
> > > code in m_config_parse_mp_command_line
> > > (near "if reauired expand DVD filename entries like dvd://1-3 into
> > > component titles")
> > > does not allow to have '-' in  "dvd://1//path/to/my/dvd/on/hdd" url.
> > > Those code (stream-specific hack, imho) should be fixed first, but i
> > > don't know clean way to do this.
> >
> > Right, that was the thing. imho the range should just be handled by the
> > stream itself. That hack in the command line parser was added as a kind
> > of poor man chapter seeking. Now that we have real chapter seeking it
> > should just be dropped, it's getting in the way.
> >
>
> Patch with suport for dvd://<title>/<full patch to dvd folder> (e.g.
> dvd://1//tmp/dvd/)
> is attached.
>
> It can be applied at any time, but without removing those hack,
> folders with '-' in path will be work.

I had been working on a patch to extend that hack to vcd and
cd-specific files, but now I realise that a specific track-seeking
code should be added to those specific streams instead. So, does
anyone object to that dvd-specific hack to be removed from
parser-mpcmd.c? If it's ok, i'll start working on track-seeking stuff
for cd and vcd media instead. This should extend the capabilities as I
wanted, but in a non-hackish way.

//Andrew