[FFmpeg-devel] [PATCH] diracdec: fix deprecated API usage

[James Almer]

On 11/2/2017 7:39 PM, Nicolas Frattaroli wrote:
> avcodec_get_chroma_sub_sample is deprecated and generates a warning
> during build, so av_pix_fmt_get_chroma_sub_sample is used
> 
> Signed-off-by: Nicolas Frattaroli <ffmpeg at fratti.ch>
> ---
>  libavcodec/diracdec.c | 12 ++++++++++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec.c
> index 0abb8b0599..7f83ed47f2 100644
> --- a/libavcodec/diracdec.c
> +++ b/libavcodec/diracdec.c
> @@ -26,6 +26,7 @@
>   * @author Marco Gerards <marco at gnu.org>, David Conrad, Jordi Ortiz <nenjordi at gmail.com>
>   */
>  
> +#include "libavutil/pixdesc.h"
>  #include "libavutil/thread.h"
>  #include "avcodec.h"
>  #include "get_bits.h"
> @@ -1927,7 +1928,10 @@ static int get_buffer_with_edge(AVCodecContext *avctx, AVFrame *f, int flags)
>  {
>      int ret, i;
>      int chroma_x_shift, chroma_y_shift;
> -    avcodec_get_chroma_sub_sample(avctx->pix_fmt, &chroma_x_shift, &chroma_y_shift);
> +    ret = av_pix_fmt_get_chroma_sub_sample(avctx->pix_fmt, &chroma_x_shift,
> +                                           &chroma_y_shift);
> +    if (ret)
> +        return ret;
>  
>      f->width  = avctx->width  + 2 * EDGE_WIDTH;
>      f->height = avctx->height + 2 * EDGE_WIDTH + 2;
> @@ -2126,7 +2130,11 @@ static int dirac_decode_data_unit(AVCodecContext *avctx, const uint8_t *buf, int
>  
>          s->pshift = s->bit_depth > 8;
>  
> -        avcodec_get_chroma_sub_sample(avctx->pix_fmt, &s->chroma_x_shift, &s->chroma_y_shift);
> +        ret = av_pix_fmt_get_chroma_sub_sample(avctx->pix_fmt,
> +                                               &s->chroma_x_shift,
> +                                               &s->chroma_y_shift);
> +        if (ret)
> +            return ret;
>  
>          ret = alloc_sequence_buffers(s);
>          if (ret < 0)

Pushed, thanks.