[MPlayer-dev-eng] [PATCH] Add "hwaccel" codecs attribute (Was: Add -va for video acceleration)

[Gwenole Beauchesne]

Hi,

On Tue, 16 Feb 2010, Gwenole Beauchesne wrote:

>>> I really don't want to duplicate videocodec entries since this is
>>> error-prone. see e.g. VDPAU, the fourcc lines now deviate a little. This
>>> could be worse when there will be 4 hwaccels to maintain...
>> 
>> I think the best way to address this would be extend codecs.conf syntax
>> somehow to avoid the duplication.
>
> That's my "hwaccel" attribute suggestion.
>
> e.g. we have
> videocodec ffmpeg12
> ...
> hwaccel vaapi
> hwaccel vdpau
>
> this would generate valid ffmpeg12{,vaapi,vdpau}. We can also have a 
> "long_name" version in the hwaccel map table to extend the name. "FFmpeg 
> MPEG-1/2 decoder (with HWACCEL)"

Here is a patch. As a side effect, aliasing -vc HWACCEL to all HWACCELs is 
rather trivial since we now have an codecs_t.hwaccel member. Though, this 
is not implemented yet.

For testing, adding just hwaccel vaapi in ffmpeg2, now generates:
$ ./mplayer -vc help|grep ffmpeg2
ffmpeg2     ffmpeg    working   FFmpeg MPEG-2  [mpeg2video]
ffmpeg2vaapi ffmpeg    working   FFmpeg MPEG-2 (VA-API)  [mpeg2video]

Regards,
Gwenole.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer.hwaccel.codec.patch
Type: text/x-diff
Size: 14502 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100217/6a308388/attachment.patch>