[FFmpeg-devel] [PATCH] support for ordered chapters/segment linking in Matroska

[Michael Niedermayer]

On Mon, Oct 27, 2008 at 04:33:44PM +0100, Aurelien Jacobs wrote:
> On Fri, 17 Oct 2008 22:57:54 +0200
> "Anton Khirnov" <wyskas at gmail.com> wrote:
> 
> > ping - a reworked version.
> 
> Only a quick review for now.
> 
> > [...]
> > +static int av_load_external_streams(AVFormatContext *s)
> > +{
> > [...]
> > +        if(!(dir = opendir(dirname)))
> > +            av_log(s, AV_LOG_ERROR, "Error opening directory %s\n.",
> > dirname);
> > +        else {
> > +            av_log(s, AV_LOG_DEBUG, "Looking for linked files in
> > %s.\n", dirname);
> > +            while((file = readdir(dir))) {
> 
> Using opendir() / readdir() directly here is ugly and won't ever
> work for non-local files.
> Another idea would be to add some kind of opendir/readdir operations
> in avio, working with URLContext. Those operations would only need
> to be implemented for local files for now, but could also be
> implemented for http or any other kind of protocol in the future.

this could be done, though it will only work when the server allows directoty
listings and our code manages to parse what it returns ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081027/49bfa50c/attachment.pgp>