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

[Michael Niedermayer]

On Mon, Feb 23, 2009 at 12:48:17PM +0100, Gwenole Beauchesne wrote:
> Hi,
>
> On Fri, 20 Feb 2009, Michael Niedermayer wrote:
>
>>> +    avctx->hwaccel = ff_query_hwaccel_codec(avctx, avctx->codec->id);
>>> +    if (avctx->hwaccel)
>>> +        avctx->pix_fmt = avctx->hwaccel->pix_fmt;
>>> +    else
>>> +        avctx->pix_fmt = PIX_FMT_YUV420P;
>>> +
>>>      /* select sub codec */
>>>      switch(avctx->codec->id) {
>>>      case CODEC_ID_H263:
>>
>> avctx->pix_fmt = ff_query_pixfmt(avctx, avctx->codec->id);
>> avctx->hwaccel = ff_pixfmt_to_hwaccal(avctx->codec->id, avctx->pix_fmt);
>
> Finally, it looks a little messier that way, IMHO. Just have a look, 
> particularly in mpeg12.c. I have attached a new patch and SVN updated but 
> haven't found any hunk committed in. So, the attached patch is still the 
> whole thing.

can you send a patch that just contains the approved hunks (minus anything
that breaks compile/regressions) ?

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

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- 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/20090223/35e702ee/attachment.pgp>