[FFmpeg-devel] [PATCH] lavf: add a concat demuxer.

[Clément Bœsch]

On Thu, Nov 22, 2012 at 07:34:15PM +0100, Nicolas George wrote:
> Le sextidi 6 fructidor, an CCXX, Nicolas George a écrit :
> > Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> > ---
> >  Changelog                |    1 +
> >  doc/demuxers.texi        |   31 +++++++
> >  libavformat/Makefile     |    1 +
> >  libavformat/allformats.c |    1 +
> >  libavformat/concatdec.c  |  221 ++++++++++++++++++++++++++++++++++++++++++++++
> >  5 files changed, 255 insertions(+)
> >  create mode 100644 libavformat/concatdec.c
> > 
> > 
> > If the basic principle is deemed sound, I intend to extend the syntax and
> > implementation, in particular to allow complete seeking in the virtual file,
> > provided the underlying files are seekable too.
> > 
> > This is not redundant with HLS because HLS does the virtual concatenation at
> > stream level, requiring the fragments to be in a concatenable format.
> > 
> > Concerning security: any kind of playlist support is potentially a security
> > concern. My intention is to only allow filenames with safe characters,
> > unless an option is explicitely enabled. Right now, since the format can not
> > be probed, it does not matter.
> 
> Hi.
> 
> Apart from Michael's suggestions of using it (in an extended version) to fix
> files with timestamps discontinuities (I believe it can work), are there
> remarks on this?
> 

As you already know, I'd really like to get this upstream soon since it's
a pretty useful feature to do a fast and lossless concat, and is often
asked by users.

The only remark I have is that I'd love to have a "repeat" mode per file
in this demuxer, so it might be interesting to define a specs for the
"playlist format" (which will also solve the probing problem you mention
in the doc).

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121122/d832a131/attachment.asc>