diff mbox

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

Message ID 1570843714-16477-3-git-send-email-hwrenx@126.com
State New
Headers show

Commit Message

hwren Oct. 12, 2019, 1:28 a.m. UTC 
From: hwrenx <hwrenx@126.com>

Remove deprecated paramete FrameRate (frame rate code) and use fps (frame rate).
Avoid encoder warning.

Signed-off-by: hwrenx <hwrenx@126.com>
---
 libavcodec/libxavs2.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Michael Niedermayer Oct. 13, 2019, 9:05 p.m. UTC | #1 
On Sat, Oct 12, 2019 at 09:28:34AM +0800, hwren wrote:
> From: hwrenx <hwrenx@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@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

[...]
diff mbox

Patch

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);
 
     cae->encoder = cae->api->encoder_create(cae->param);
     if (!cae->encoder) {