[FFmpeg-devel] [PATCH 4/7] avcodec/smvjpegdec: return the packet size instead of 0

[Paul B Mahol]

On 12/15/16, Michael Niedermayer <michael at niedermayer.cc> wrote:
> Most decoders return the amount of data used.
> This is more consistent
>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/smvjpegdec.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/smvjpegdec.c b/libavcodec/smvjpegdec.c
> index e319e5781b..018e135b30 100644
> --- a/libavcodec/smvjpegdec.c
> +++ b/libavcodec/smvjpegdec.c
> @@ -200,9 +200,11 @@ static int smvjpeg_decode_frame(AVCodecContext *avctx,
> void *data, int *data_siz
>              s->picture[1]->linesize[i] = mjpeg_data->linesize[i];
>
>          ret = av_frame_ref(data, s->picture[1]);
> +        if (ret < 0)
> +            return ret;
>      }
>
> -    return ret;
> +    return avpkt->size;
>  }
>
>  static const AVClass smvjpegdec_class = {
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

lgtm