[MPlayer-dev-eng] [PATCH] add VDPAU IMGFMTs

[Gwenole Beauchesne]

Hi,

Le 12 févr. 09 à 18:52, Reimar Döffinger a écrit :

> It adds the IMGFMTs needed, and only those (so that there is a 1:1
> correspondence to the lavc ones, not split by profile).

There should be a means to factorize the duplicated vcodecs for vdpau  
(et al.).

I'd suggest adding a single out "VPDAU_foo". However, this approach  
has a problem: some codecs are marked as status working and buggy and  
could be working with VDPAU but buggy with the original SW decoder.

However, I'd then suggest a new section "hwaccel" that receives  
status, names, etc. Benefits:
- Can implement some -va help very easily by listing the accepted  
accelerators
- The above mentionned problem can vanish in the matching function by  
"if hwaccel available and status working then consider this codec as  
working"
- The "out VDPAU_foo" will become "hwaccel VDPAU_foo"
- From a user point of view, having to specify -vc codec_vdpau is  
counter intuitive, a simple -va hwaccel would work better for  
configurations where you mix input video types. e.g. play mpeg2 then  
h264, no need to torture yourself with -vc args.

WDYT?

Regards,
Gwenole.