Re: [FFmpeg-devel] [PATCH 09/13] lavc/videotoolboxenc: Add keyframes_only property

2016-04-10 Thread Hendrik Leppkes
On Sun, Apr 10, 2016 at 6:29 AM, Carl Eugen Hoyos wrote: > Rick Kern gmail.com> writes: > >> +{ "keyframes_only", "Output only I-frames", OFFSET(keyframes_only), > > I suspect this should check for -g 1 instead. > Indeed, checking avctx->gop_size would make it compatible with other encoders

Re: [FFmpeg-devel] [PATCH 09/13] lavc/videotoolboxenc: Add keyframes_only property

2016-04-09 Thread Carl Eugen Hoyos
Rick Kern gmail.com> writes: > +{ "keyframes_only", "Output only I-frames", OFFSET(keyframes_only), I suspect this should check for -g 1 instead. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/

[FFmpeg-devel] [PATCH 09/13] lavc/videotoolboxenc: Add keyframes_only property

2016-04-09 Thread Rick Kern
Outputs only I-frames when set. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 5430479..c9a60b7 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcode