[FFmpeg-cvslog] r20502 - in trunk: configure libavcodec/allcodecs.c libavcodec/h263dec.c libavcodec/imgconvert.c libavcodec/vdpau.c libavcodec/vdpau.h libavcodec/vdpau_internal.h libavutil/pixfmt.h

[Uoti Urpala]

On Tue, 2009-11-10 at 19:52 +0100, cehoyos wrote:
> Author: cehoyos
> Date: Tue Nov 10 19:52:39 2009
> New Revision: 20502
> 
> Log:
> Add VDPAU hardware accelerated decoding for MPEG-4 ASP which can be used
> by video players.

The configure test still hasn't been fixed.

Also this doesn't look safe:

> Modified: trunk/libavcodec/vdpau.h
> ==============================================================================
> --- trunk/libavcodec/vdpau.h	Tue Nov 10 16:56:03 2009	(r20501)
> +++ trunk/libavcodec/vdpau.h	Tue Nov 10 19:52:39 2009	(r20502)
> @@ -73,6 +73,7 @@ struct vdpau_render_state {
>          VdpPictureInfoH264     h264;
>          VdpPictureInfoMPEG1Or2 mpeg;
>          VdpPictureInfoVC1       vc1;
> +        VdpPictureInfoMPEG4Part2 mpeg4;
>      } info;

If adding this to the union doesn't break ABI in practice it's only
because the mpeg4 struct happens to be smaller than the previous ones.
What about future additions? Using a pointer would be safer.