[FFmpeg-devel] [PATCH] lavc/qsvenc: provide detail error message if parameters invalid
zhong.li at intel.com
Tue Mar 13 14:11:32 EET 2018
> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf
> Of Moritz Barsnick
> Sent: Tuesday, March 13, 2018 6:13 PM
> To: FFmpeg development discussions and patches
> <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: provide detail error
> message if parameters invalid
> On Fri, Mar 02, 2018 at 11:24:10 +0800, Zhong Li wrote:
> > ret = MFXVideoENCODE_Query(q->session, &q->param,
> > - if (ret < 0 ||
> > - param_out.mfx.RateControlMethod !=
> This original code gave the impression that a mismatch of
> RateControlMethod was not reported by MFXVideoENCODE_Query(), and
> therefore checked separately. You are not doing that anymore.
That is an intentional behavior. If current bit rate control mode is not supported. MFXVideoENCODE_Query() should return an invalid value as MSDK documentation.
> > + if (UNMATCH(FrameInfo.FrameRateExtN) ||
> > + UNMATCH(FrameInfo.FrameRateExtN))
> Typo? (Both macro arguments are identical.)
Yes, it is should be "FrameInfo.FrameRateExtD", I have a patch to fix it and will send soon.
More information about the ffmpeg-devel