Quoting John Stebbins (2015-10-28 17:48:34) > From: Stefano Sabatini <[email protected]> > > (cherry picked from ffmpeg commit 7cd5fa35fecc2717223459d04822faed6bbd40fa) > --- > libavfilter/vsrc_testsrc.c | 15 +++++++-------- > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c > index e41625e..cb13d7e 100644 > --- a/libavfilter/vsrc_testsrc.c > +++ b/libavfilter/vsrc_testsrc.c > @@ -46,7 +46,7 @@ typedef struct TestSourceContext { > const AVClass *class; > int h, w; > unsigned int nb_frame; > - AVRational time_base; > + AVRational time_base, frame_rate; > int64_t pts, max_pts; > char *size; ///< video frame size > char *rate; ///< video frame rate > @@ -75,7 +75,6 @@ static const AVOption testsrc_options[] = { > static av_cold int init_common(AVFilterContext *ctx) > { > TestSourceContext *test = ctx->priv; > - AVRational frame_rate_q; > int64_t duration = -1; > int ret = 0; > > @@ -84,8 +83,8 @@ static av_cold int init_common(AVFilterContext *ctx) > return ret; > } > > - if ((ret = av_parse_video_rate(&frame_rate_q, test->rate)) < 0 || > - frame_rate_q.den <= 0 || frame_rate_q.num <= 0) { > + if ((ret = av_parse_video_rate(&test->frame_rate, test->rate)) < 0 || > + test->frame_rate.den <= 0 || test->frame_rate.num <= 0) {
This looks unnecessary, av_parse_video_rate() already checks that the value is a valid framerate. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
