[FFmpeg-devel] [PATCH 2/2] lavf/dump: hide a few deprecation warnings.

James Almer jamrial at gmail.com
Sun Dec 8 15:21:06 EET 2019


On 12/8/2019 8:47 AM, Nicolas George wrote:
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>  libavformat/dump.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/libavformat/dump.c b/libavformat/dump.c
> index 56814ff7d2..fcf8bad63a 100644
> --- a/libavformat/dump.c
> +++ b/libavformat/dump.c
> @@ -479,12 +479,14 @@ static void dump_stream_format(AVFormatContext *ic, int i,
>      }
>  
>      // Fields which are missing from AVCodecParameters need to be taken from the AVCodecContext
> +FF_DISABLE_DEPRECATION_WARNINGS
>      avctx->properties = st->codec->properties;
>      avctx->codec      = st->codec->codec;
>      avctx->qmin       = st->codec->qmin;
>      avctx->qmax       = st->codec->qmax;
>      avctx->coded_width  = st->codec->coded_width;
>      avctx->coded_height = st->codec->coded_height;
> +FF_ENABLE_DEPRECATION_WARNINGS
>  
>      if (separator)
>          av_opt_set(avctx, "dump_separator", separator, 0);
> @@ -519,7 +521,9 @@ static void dump_stream_format(AVFormatContext *ic, int i,
>          int fps = st->avg_frame_rate.den && st->avg_frame_rate.num;
>          int tbr = st->r_frame_rate.den && st->r_frame_rate.num;
>          int tbn = st->time_base.den && st->time_base.num;
> +FF_DISABLE_DEPRECATION_WARNINGS
>          int tbc = st->codec->time_base.den && st->codec->time_base.num;
> +FF_ENABLE_DEPRECATION_WARNINGS
>  
>          if (fps || tbr || tbn || tbc)
>              av_log(NULL, AV_LOG_INFO, "%s", separator);
> @@ -530,8 +534,10 @@ static void dump_stream_format(AVFormatContext *ic, int i,
>              print_fps(av_q2d(st->r_frame_rate), tbn || tbc ? "tbr, " : "tbr");
>          if (tbn)
>              print_fps(1 / av_q2d(st->time_base), tbc ? "tbn, " : "tbn");
> +FF_DISABLE_DEPRECATION_WARNINGS
>          if (tbc)
>              print_fps(1 / av_q2d(st->codec->time_base), "tbc");
> +FF_ENABLE_DEPRECATION_WARNINGS

All these should be wrapped in checks for FF_API_LAVF_AVCTX as well.

LGTM otherwise.

>      }
>  
>      if (st->disposition & AV_DISPOSITION_DEFAULT)
> 



More information about the ffmpeg-devel mailing list