[FFmpeg-devel] [PATCH] lavf/segment: add -segment_delta option

[Stefano Sabatini]

On date Sunday 2012-07-08 17:48:59 +0200, Stefano Sabatini encoded:
> On date Saturday 2012-07-07 17:53:50 +0200, Michael Niedermayer encoded:
> > On Wed, Jul 04, 2012 at 01:50:55AM +0200, Stefano Sabatini wrote:
> [...]
> > >  @table @option
> > > + at item segment_delta @var{delta}
> > > +Specify the accuracy in seconds when selecting the start time for a
> > > +segment. Default value is "0".
> > > +
> > > +This is especially useful with video segments, which are split at GOP
> > > +boundaries. When delta is specified a given key-frame is selected if
> > > +its PTS satisfies the relation:
> > > + at example
> > > +PTS >= start_time - delta
> > > + at end example
> > 
> > this description is hard to understand, otherwise patch LGTM
> > 
> > Maybe it would be clearer if the problem segment_delta solves would
> > be described, or maybe the effect of a larger or small delta could
> > be described
> 
> Added a more extensive explanation, together with a complete example.
> -- 
> FFmpeg = Friendly and Faithless Moronic Proud Elastic Geek

> From 73e91a6ea22a5b91508a940066fc7c24441e11a9 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Sat, 28 Jan 2012 22:21:43 +0100
> Subject: [PATCH] lavf/segment: add -segment_time_delta option
> 
> FIXME: bump micro version
> ---
>  doc/muxers.texi       |   30 ++++++++++++++++++++++++++++++
>  libavformat/segment.c |   14 +++++++++++++-
>  2 files changed, 43 insertions(+), 1 deletions(-)

Applied.
-- 
FFmpeg = Formidable and Formidable Mysterious Pure Elaborated Gorilla