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

[Diego Biurrun]

On Sat, Jan 03, 2009 at 09:49:30AM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > > --- libavcodec/vdpauvideo.c	(revision 0)
> > > +++ libavcodec/vdpauvideo.c	(revision 0)
> > > @@ -0,0 +1,199 @@
> > > +static void VDPAU_ensure_has_buffers(vdpau_render_state * render,
> > > +        int need_entries)
> > 
> > weird indentation
> 
> I copied it from aac.c.

That file could probably use some beautification.

> Please tell me how to do it correctly.

static void VDPAU_ensure_has_buffers(vdpau_render_state * render,
                                     int need_entries)

> Please note that it is not a good idea to put the declarations in vdpau_render.h
> because you cannot compile mplayer like this.
> I did move it back to h264.c, but I'm open for every suggestion. (an additional
> header?)

vdpau_render.h would need to become an installed header, the same goes
for xvmc_render.h.  However, Ivan has been talking about making some
changes to it before it can become an installed header.  This was years
ago, Godot is yet to arrive...

Diego