On Wed, Feb 25, 2009 at 05:19:56PM +0100, Gwenole Beauchesne wrote: > On Tue, 24 Feb 2009, Michael Niedermayer wrote: > > >>>> @@ -616,6 +630,11 @@ retry: > >>>> if(MPV_frame_start(s, avctx) < 0) > >>>> return -1; > >>>> > >>>> + if (avctx->hwaccel) { > >>>> + if (avctx->hwaccel->start_frame(avctx, s->gb.buffer, s- > >>>>> gb.size_in_bits/8) < 0) > >>>> + return -1; > >>>> + } > >>>> + > >>>> #ifdef DEBUG > >>>> av_log(avctx, AV_LOG_DEBUG, "qscale=%d\n", s->qscale); > >>>> #endif > >>> > >>> its buf/buf_size > >> > >> even when we get here again through the retry: label? > > > > i think so > > Really? Why is init_get_bits() used with either bitstream_buffer or buf > arg? It is buf/buf_size in case of a goto retry if you want to support the malfomed mpeg4 some versions of divx generate then your original code might work with some luck though still the values, especially the buffer size would be wrong i think > Is there a sample exhibiting this divx bitstream reorder stuff?: <>