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

Stefano Sabatini stefasab at gmail.com
Tue Jan 22 01:16:13 CET 2013


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.
-- 
FFmpeg = Fostering and Forgiving Magnificient Power Embarassing Gospel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ffmpeg-implement-force_key_frames-expression-evaluti.patch
Type: text/x-diff
Size: 8467 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130122/bd3eae6a/attachment.bin>


More information about the ffmpeg-devel mailing list