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

[Gwenole Beauchesne]

On Tue, 24 Feb 2009, Michael Niedermayer wrote:

>>> Now, your suggested approach requires as many lists as (HW accelerators) x
>>> (chroma formats) x (sub-codecs).
>>
>> You dont need to split pix_fmts per codec, its done currently and iam not
>> asking you to change it but i would be happy if you did change it :)
>
> after seeing your patch, i think per codec pix_fmts really should be
> removed.
> This would make the code much simpler and cleaner. It also would greatly
> simplify get_format() for user apps,having to test just for one pix_fmt
> er hwaccel API instead of for one per codec X API

Not really. The user app then would have to check avctx->codec->id itself 
as not all accelerators would implement the same codec...