[FFmpeg-devel] [PATCH]Do not allow DVCPRO HD encoding, it is broken

Carl Eugen Hoyos cehoyos at ag.or.at
Mon Jun 4 20:43:34 CEST 2012


Michael Niedermayer <michaelni <at> gmx.at> writes:

> > > > +    if (avctx->coded_height > 576 ||
> > > > +        !avctx->coded_height && avctx->height > 576) {
> > > 
> > > i dont think that 2 checks are needed
> > 
> > I copied that from avpriv_dv_codec_profile(), which check
> 
> thats used for decoders too.where lowres could lead to differences

Thank you for the explanation!

> > can be removed?
> 
> coded_height

Patch applied.

Thank you, Carl Eugen



More information about the ffmpeg-devel mailing list