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

[Michael Niedermayer]

On Thu, Dec 15, 2016 at 09:35:25PM +0100, Paul B Mahol wrote:
> On 12/15/16, Michael Niedermayer <mich...@niedermayer.cc> wrote:
> > Most decoders return the amount of data used.
> > This is more consistent
> >
> > Signed-off-by: Michael Niedermayer <mich...@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@ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> lgtm

applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.

signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel