[FFmpeg-devel] [PATCH] h264 predict functions

Michael Niedermayer michaelni
Wed Dec 29 04:12:28 CET 2010

On Tue, Dec 28, 2010 at 11:38:41AM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> > > Patch applied by Ronald in r26087 (2010-12-24).
> > 
> > people should reply to applied patches ideally with a "applied" ...
> Thank you for reminding us;-)
> Since you are currently not too busy,

iam always busy

> could you comment on why muxing
> high-bitrate (448k and higher) stereo ac3 in avi seems problematic for ffmpeg
> (ffplay and mplayer don't play the result without A/V desync)?

 blkalign = enc->frame_size; //this is wrong, but it seems many demuxers do not work if this is set correctly

make it 0 and it will work on non broken demuxers (it probably will break
microsofts own demuxers though)
there is no non 0 value that is correct for some ac3 configurations :)
but large values might work
we can try to put some random large value in there or we could try 0 for streams
of too high a bitrate.

Is someone volunteering to test this against many demuxers?

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101229/71c89d27/attachment.pgp>

More information about the ffmpeg-devel mailing list