[FFmpeg-devel] [PATCH 3/3] lavc/libxavs2: replace 'FrameRate' with 'fps'

Michael Niedermayer michael at niedermayer.cc
Mon Oct 14 00:05:25 EEST 2019


On Sat, Oct 12, 2019 at 09:28:34AM +0800, hwren wrote:
> From: hwrenx <hwrenx at 126.com>
> 
> Remove deprecated paramete FrameRate (frame rate code) and use fps (frame rate).
> Avoid encoder warning.

theres at least one spelling error in here (paramete)


> 
> Signed-off-by: hwrenx <hwrenx at 126.com>
> ---
>  libavcodec/libxavs2.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavcodec/libxavs2.c b/libavcodec/libxavs2.c
> index 47ba74d..04f4ecb 100644
> --- a/libavcodec/libxavs2.c
> +++ b/libavcodec/libxavs2.c
> @@ -113,8 +113,7 @@ static av_cold int xavs2_init(AVCodecContext *avctx)
>          xavs2_opt_set2("InitialQP", "%d", cae->qp);
>      }
>  
> -    ff_mpeg12_find_best_frame_rate(avctx->framerate, &code, NULL, NULL, 0);
> -    xavs2_opt_set2("FrameRate", "%d", code);
> +    xavs2_opt_set2("fps", "%d", avctx->framerate);

Theres a unused variable after the change

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191013/0441c3e2/attachment.sig>


More information about the ffmpeg-devel mailing list