On Sun, 2021-06-06 at 22:56 +0800, Linjie Fu wrote:
> Hi Haihao,
>
> On Mon, May 17, 2021 at 11:30 AM Haihao Xiang <haihao.xi...@intel.com> wrote:
> >
> > ---
> > libavfilter/vf_vpp_qsv.c | 8 ++++++--
> > 1 file changed, 6 insertions(+), 2 deletions(-)
> >
> > diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
> > index 29ba220665..ec35f85b04 100644
> > --- a/libavfilter/vf_vpp_qsv.c
> > +++ b/libavfilter/vf_vpp_qsv.c
> > @@ -255,10 +255,14 @@ static int config_input(AVFilterLink *inlink)
> > int ret;
> > int64_t ow, oh;
> >
> > - if (vpp->framerate.den == 0 || vpp->framerate.num == 0)
> > + /* Ignore user's setting for framerate when deinterlacing is used */
> > + if (vpp->deinterlace)
> > + vpp->framerate = av_mul_q(inlink->frame_rate,
> > + (AVRational){ 2, 1 });
> > + else if (vpp->framerate.den == 0 || vpp->framerate.num == 0)
> > vpp->framerate = inlink->frame_rate;
>
> No objection, just considering would it be better to prompt a warning
> for users instead of just ignoring the user 's setting?
Thanks for the comment, I will update it.
>
> - linjie
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
