[FFmpeg-devel] [PATCH]VDPAU patch for h264 decoding, round 7

[Diego Biurrun]

On Mon, Jan 05, 2009 at 12:07:25AM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > > Attached is a hopefully acceptable version of the libavcodec (and  
> > > libavutil) part of the H264_VDPAU decoding patch.
> > >
> > > Please comment, Carl Eugen
> > 
> > You should add build system support for enabling this as a configure
> > parameter.  Compare what I recently did for xvmc.
> > 
> > But there is an issue we need to address: How is this supposed to be
> > used?  Will it need vdpau_render.h as an installed header?
> 
> I originally thought VDPAU will only be usable from inside MPlayer (built with
> static libavcodec), but I see now that is of course also possible to install
> vdpau_render.h and use VDPAU from a shared libavcodec.
> But that would mean that vdpau_render.h is also installed if 
> /usr/include/vdpau/vdpau.h was not available when the library was built, so the
> header is unusable, because the installed library does not support vdpau.
> 
> Would that be acceptable? Or would it be acceptable if vdpau_render.h is only
> installed if the library actually supports VDPAU?

You could check for the presence of the header...

> I'm working on a configure patch and will post it soon: My problem is currently
> that it doesn't die if a dependency is not fulfilled, but just silently disables
> the function.

That should be OK, just do the same that is done for xvmc.

Diego