[FFmpeg-devel] [PATCH] Don't let ctx->skip_frame>0 produce errors
Carl Eugen Hoyos
Thu Aug 30 22:51:16 CEST 2007
On 2007-08-30 19:16, Carl Eugen Hoyos wrote:
> > > > well but what if
> > > > avctx->skip_frame>0
> > > > and this code is reached for a reason different then skip_frame ?
> > >
> > > Then the user silenced an error by disabling complete decoding.
> > > Or is there a better solution? More different return values for
> > > decode_nal_units() only make the code messier, don't you think?
> > also
> > if( (s->hurry_up == 1 && h->nal_ref_idc == 0) //FIXME do not discard SEI id
> > ||(avctx->skip_frame >= AVDISCARD_NONREF && h->nal_ref_idc == 0))
> > continue;
> > your code hides the warning only in one of the 2 cases
> I was under the obviously wrong impression that deprecated features are
> not considered when doing such a change.
And now a hopefully fixed version, Carl Eugen
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel