[FFmpeg-cvslog] r17555 - trunk/ffmpeg.c

[Michael Niedermayer]

On Tue, Feb 24, 2009 at 04:13:28PM +0100, Diego Biurrun wrote:
> On Tue, Feb 24, 2009 at 04:06:17PM +0100, michael wrote:
> > 
> > Log:
> > Do not duplicate frames if the muxer claims that its not needed.
> > Note to release manager, this is risky and should be reverted if it causes any problem.
> 
> Then why did you not wait a few more days with this?

Because we need it for fixing h.264 timestamps besides it fixes the
issue with duplicated frames where they can be avoided.

i dont mind reverting this, if you want
but reverting means no complete h264 timestamp fix in the release and
frames will still be duplicated in cases of variable fps even when unneeded.

and besides -vsync 1 on the command line reverts the effect of this commit
at runtime

[..]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090224/a60a71e8/attachment.pgp>