[FFmpeg-devel] [PATCH] mpeg12.c: add HW acceleration hooks (take 2)

Michael Niedermayer michaelni
Thu Feb 26 12:39:14 CET 2009


On Thu, Feb 26, 2009 at 10:32:19AM +0100, Gwenole Beauchesne wrote:
> Hi,
>
> On Thu, 26 Feb 2009, Michael Niedermayer wrote:
>
>>>> iam still investigating the need for this mess
>>>
>>> (i) determine the end of the slice (and thus its size), (ii) slice
>>> vertical position. Granted, for (ii), only s->mb_y = mb_y; is useful.
>>> Would the hunk be ok with the others ripped off?
>>
>> yes
>
> Here is the new patch.

[...]
> @@ -2403,6 +2421,13 @@ static int decode_chunks(AVCodecContext *avctx,
>                      break;
>                  }
>  
> +                if (avctx->hwaccel) {
> +                    ret = mpeg_decode_slice(s, mb_y, &buf_ptr, input_size);
> +                    if (ret != DECODE_SLICE_OK)
> +                        return -1;
> +                    break;
> +                }

why is the call to mpeg_decode_slice a few lines latr in the else
not enough?

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20090226/2e1a228c/attachment.pgp>



More information about the ffmpeg-devel mailing list