[FFmpeg-devel] [PATCH] HWAccel infrastructure (take 7.3)

[Michael Niedermayer]

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?

probably ...

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

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090225/b998187c/attachment.pgp>