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

[Diego Biurrun]

On Sat, Jan 03, 2009 at 12:52:09AM +0100, Carl Eugen Hoyos wrote:
>
> I tried to follow Diegos suggestions.
>
> Please comment, Carl Eugen
>
> --- 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

> +void ff_VDPAU_h264_add_data_chunk(H264Context *h, const uint8_t *buf, int buf_size)

long line

> --- libavcodec/mpegvideo.c	(revision 16386)
> +++ libavcodec/mpegvideo.c	(working copy)
> @@ -957,7 +957,7 @@
> -    if(s->unrestricted_mv && s->current_picture.reference && !s->intra_only && !(s->flags&CODEC_FLAG_EMU_EDGE)) {
> +    if(!s->avctx->vdpau_acceleration && s->unrestricted_mv && s->current_picture.reference && !s->intra_only && !(s->flags&CODEC_FLAG_EMU_EDGE)) {

You could break this line while you're at it..

> --- libavcodec/vdpau_render.h	(revision 0)
> +++ libavcodec/vdpau_render.h	(revision 0)
> @@ -0,0 +1,93 @@
> +
> +typedef struct{
> + [...]
> +} vdpau_render_state;

This is (now) done without typedef in xvmc_render.h.

Diego