[FFmpeg-devel] [PATCH]VDPAU patch for MPEG1/2 decoding, round 1

Carl Eugen Hoyos cehoyos
Mon Jan 5 19:38:08 CET 2009


Reimar D?ffinger <Reimar.Doeffinger <at> stud.uni-karlsruhe.de> writes:

> > +static void mpeg_set_pixelformat(AVCodecContext *avctx){
> > +    Mpeg1Context *s1 = avctx->priv_data;
> > +    MpegEncContext *s = &s1->mpeg_enc_ctx;
> > +
> > +    if(avctx->codec->capabilities&CODEC_CAP_HWACCEL_VDPAU){
> 
> IMO CODEC_CAP_HWACCEL_VDPAU should not be used at all, checking codec_id
> seems like the sanest way to me currently.

The problem is that mpeg12.c sets codec_id three times rather unconditionally to
CODEC_ID_MPEGnVIDEO: lines 1399, 2043, 2104

I'm not sure if there is a reason for those assignments or if they can be just
removed. Could anybody enlighten me?

Carl Eugen






More information about the ffmpeg-devel mailing list