[FFmpeg-cvslog] r16431 - in trunk: configure libavcodec/Makefile libavcodec/allcodecs.c libavcodec/avcodec.h libavcodec/h264.c libavcodec/h264_parser.c libavcodec/imgconvert.c libavcodec/mpegvideo.c libavcodec/vdp...

[Diego Biurrun]

On Mon, Jan 05, 2009 at 12:55:28AM +0100, cehoyos wrote:
> 
> Log:
> Add VDPAU hardware accelerated decoding for H264 which will be used by
> MPlayer.
> 
> --- /dev/null	00:00:00 1970	(empty, because file is newly added)
> +++ trunk/libavcodec/vdpau_render.h	Mon Jan  5 00:55:27 2009	(r16431)
> @@ -0,0 +1,86 @@
> +
> + *
> + * As per the current implementation, the actual decoding
> + * and rendering (API calls) are done as part of VDPAU
> + * presentation (vo_vdpau.c) module.
> + *

> + * \brief This structure is used as a CALL-BACK between the ffmpeg
> + * decoder (vd_) and presentation (vo_) module.
> + * This is used for defining a video-frame containing surface,
> + * picture-parameter, bitstream informations etc which are passed
> + * between ffmpeg decoder and its clients.

These MPlayer-specific comments are still present.

> +struct vdpau_render_state
> +    VdpVideoSurface surface; ///< used as rendered surface, never changed.
> +
> +    int state; ///< Holds FF_VDPAU_STATE_* values
> +
> +    /** Picture Parameter information for all supported codecs */
> +    union _VdpPictureInfo {
> +        VdpPictureInfoH264     h264;
> +    } info;
> +
> +    /** Describe size/location of the compressed video data */
> +    int bitstreamBuffersAlloced;
> +    int bitstreamBuffersUsed;
> +    VdpBitstreamBuffer *bitstreamBuffers;

This still uses ugly camel-cased variable names.

Can we please address these issues before proceeding to the next
steps of vdpau support?

Diego