On 06/02/17 12:33, Tobias Rapp wrote: > Sets framerate field in output codec context if explicitly specified on > the command-line. > > Signed-off-by: Tobias Rapp <t.r...@noa-archive.com> > --- > ffmpeg_opt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > index 6a47d32..3b532da 100644 > --- a/ffmpeg_opt.c > +++ b/ffmpeg_opt.c > @@ -1535,6 +1535,8 @@ static OutputStream *new_video_stream(OptionsContext > *o, AVFormatContext *oc, in > av_log(NULL, AV_LOG_FATAL, "Invalid framerate value: %s\n", > frame_rate); > exit_program(1); > } > + if (frame_rate && ost->frame_rate.num && ost->frame_rate.den) > + video_enc->framerate = ost->frame_rate; > if (frame_rate && video_sync_method == VSYNC_PASSTHROUGH) > av_log(NULL, AV_LOG_ERROR, "Using -vsync 0 and -r can produce > invalid output files\n"); > >
Is there a reason not to always set this, rather than only when it is specified explicitly on the command line as you have? (Like <https://git.libav.org/?p=libav.git;a=commit;h=d10102d23c9467d4eb84f58e0cd12be284b982f6>, though that is after the current merge point and I don't know if it interacts with anything else.) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
