[FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg_opt: hide a few deprecation warnings.

James Almer jamrial at gmail.com
Sun Dec 8 15:17:48 EET 2019


On 12/8/2019 8:47 AM, Nicolas George wrote:
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>  fftools/ffmpeg_opt.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
> index 71063cc443..5ce70db23e 100644
> --- a/fftools/ffmpeg_opt.c
> +++ b/fftools/ffmpeg_opt.c
> @@ -34,6 +34,7 @@
>  #include "libavutil/avstring.h"
>  #include "libavutil/avutil.h"
>  #include "libavutil/channel_layout.h"
> +#include "libavutil/internal.h"
>  #include "libavutil/intreadwrite.h"
>  #include "libavutil/fifo.h"
>  #include "libavutil/mathematics.h"
> @@ -796,6 +797,7 @@ static void add_input_streams(OptionsContext *o, AVFormatContext *ic)
>              if(!ist->dec)
>                  ist->dec = avcodec_find_decoder(par->codec_id);
>  #if FF_API_LOWRES
> +FF_DISABLE_DEPRECATION_WARNINGS
>              if (st->codec->lowres) {
>                  ist->dec_ctx->lowres = st->codec->lowres;
>                  ist->dec_ctx->width  = st->codec->width;
> @@ -803,6 +805,7 @@ static void add_input_streams(OptionsContext *o, AVFormatContext *ic)
>                  ist->dec_ctx->coded_width  = st->codec->coded_width;
>                  ist->dec_ctx->coded_height = st->codec->coded_height;
>              }
> +FF_ENABLE_DEPRECATION_WARNINGS
>  #endif
>  
>              // avformat_find_stream_info() doesn't set this for us anymore.
> @@ -2398,9 +2401,12 @@ loop_end:
>  
>          if ((ost->stream_copy || ost->attachment_filename)
>              && (e = av_dict_get(o->g->codec_opts, "flags", NULL, AV_DICT_IGNORE_SUFFIX))
> -            && (!e->key[5] || check_stream_specifier(oc, ost->st, e->key+6)))
> +            && (!e->key[5] || check_stream_specifier(oc, ost->st, e->key+6))) {
> +FF_DISABLE_DEPRECATION_WARNINGS
>              if (av_opt_set(ost->st->codec, "flags", e->value, 0) < 0)
>                  exit_program(1);
> +FF_ENABLE_DEPRECATION_WARNINGS

Last time i tried to silence deprecation warnings on fftool/* files it
broke compilation in some cases/compilers, since apparently the
__pragma() these expand into would clash with something else. I don't
recall what compiler, though.

> +        }
>      }
>  #endif
>  
> 



More information about the ffmpeg-devel mailing list