On Thu, Nov 24, 2011 at 01:54:50PM +0000, Måns Rullgård wrote:
> Anton Khirnov <[email protected]> writes:
> 
> > From: Reimar Döffinger <[email protected]>
> >
> > Otherwise it might return a write-only frame which would break
> > decoding completely.
> 
> What does that mean?

That you'll get garbage instead of frame with data from previous decode.
 
> > Signed-off-by: Anton Khirnov <[email protected]>
> > ---
> >  libavcodec/roqvideodec.c |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavcodec/roqvideodec.c b/libavcodec/roqvideodec.c
> > index 64a15fa..527ba51 100644
> > --- a/libavcodec/roqvideodec.c
> > +++ b/libavcodec/roqvideodec.c
> > @@ -175,6 +175,7 @@ static int roq_decode_frame(AVCodecContext *avctx,
> >      RoqContext *s = avctx->priv_data;
> >      int copy= !s->current_frame->data[0];
> >  
> > +    s->current_frame->reference = 3;
> 
> What is the meaning of 3?

"both fields are used for reference" or so doxy in avcodec.h says.

Am I the only one who finds this system unintuitive?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to