[FFmpeg-devel] [PATCH] ffmpeg: implement -force_key_frames_expr option

[Stefano Sabatini]

On date Tuesday 2013-01-22 01:16:13 +0100, Stefano Sabatini encoded:
> On date Sunday 2013-01-13 22:48:17 +0100, Nicolas George encoded:
> > Le quartidi 24 nivôse, an CCXXI, Stefano Sabatini a écrit :
> > > Also, after some thinking I'm not sure it is a good idea to "pollute"
> > > libavcodec (and I can't find a simple way to implement this at the
> > > library level), and I don't want to churn library code more than
> > > necessary. Thus basically I'm reproposing this patch in the current
> > > form, which fixes the problem for me.
> > 
> > I think I agree with that position.
> > 
> > > A possible alternative to the current approach (which computes the
> > > time value for the next split) would be to evaluate an expression for
> > > every frame, so that for example:
> > > expr:n*10
> > > 
> > > would become:
> > > expr:gte(t, n*10)
> > > 
> > > which would be more flexible.
> > 
> > If there are enough available variables, then this is better indeed.
> 
> Updated.
> 
> Note: an ifelse() ternary function would simplify syntax.

Updated. I'll push in three days if I read no comments.
-- 
FFmpeg = Fascinating and Freak Monstrous Philosofic Exxagerate Gangster
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ffmpeg-implement-force_key_frames-expression-evaluti.patch
Type: text/x-diff
Size: 9062 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130124/6bad3ceb/attachment.bin>