[MPlayer-dev-eng] VA API support for MPlayer

Diego Biurrun diego at biurrun.de
Mon Jan 5 12:03:24 CET 2009


On Mon, Jan 05, 2009 at 11:34:09AM +0100, Gwenole Beauchesne wrote:
> 
> On Wed, 24 Dec 2008, Diego Biurrun wrote:
> 
> > Please use our standard license header instead.
> 
> Well, I looked for a good looking license header, and I reached to the
> one from vo_wii.c. I will grab another one then.

vo_wii.c never had such a license header.  Whatever, just use the
standard one.

> >> +typedef struct {
> >> +  VADisplay display;
> >> +  VAConfigID config_id;
> >> +  VAContextID context_id;
> >> +} vaapi_context_t;
> > 
> > The _t namespace is reserved by POSIX, use another name, possibly
> > drop the typedef.
> 
> Would something like VAAPI_Context be acceptable? i.e. VAAPI_ prefix + 
> camel-case typename? I'd prefer sticking to lower-case for identifiers and 
> use something different for type names.

I think a better idea would be to drop the typedef.

Diego



More information about the MPlayer-dev-eng mailing list