[FFmpeg-devel] [PATCH] lavf: add a concat demuxer.
ubitux at gmail.com
Thu Nov 22 22:27:31 CET 2012
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.
> 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).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 490 bytes
Desc: not available
More information about the ffmpeg-devel