[FFmpeg-devel] [PATCH] Native VP9 decoder.

[Ronald S. Bultje]

Hi,

On Tue, Oct 1, 2013 at 2:31 PM, Michael Niedermayer <michaelni at gmx.at>wrote:

> On Mon, Sep 30, 2013 at 11:03:17PM -0400, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Mon, Sep 30, 2013 at 8:15 AM, Derek Buitenhuis <
> > derek.buitenhuis at gmail.com> wrote:
> [...]
>
> > > > +static int vp9_decode_packet(AVCodecContext *avctx, void *out_pic,
> > > > +                             int *got_frame, AVPacket *avpkt)
> > > > +{
> > >
> > > [...]
> > >
> > > > +    // read superframe index - this is a collection of individual
> > > frames that
> > > > +    // together lead to one visible frame
> > > > +    if (size <= 0)
> > > > +        return AVERROR_INVALIDDATA;
> > >
> > > Is it not guaranteed to have a non-zero packet?
> > >
> >
> > Not sure, Michael?
>
> without CODEC_CAP_DELAY 0 sized packets are currently not passed to
> the decoder.
> I dont know if that will always be true in the future
>

OK changed locally into an av_assert1. I'll push to my github repository,
no need for a new patch on NL at this point I think.

Ronald