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

Michael Niedermayer michaelni at gmx.at
Mon Jun 4 20:33:57 CEST 2012


On Mon, Jun 04, 2012 at 06:22:16PM +0000, Carl Eugen Hoyos wrote:
> 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


> can be removed?

coded_height

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120604/319a17b6/attachment.asc>


More information about the ffmpeg-devel mailing list