[MPlayer-dev-eng] [PATCH] delay-compensated B-frames and dwStart

Corey Hickey bugfood-ml at fatooh.org
Tue Feb 14 00:25:09 CET 2006


Nico Sabbi wrote:
>>That's just fine. Actually, the main thing to test is to make sure I
>>didn't break the current seeking functionality by failing to find and
>>alter any instances of demux_seek().
> 
> please, before committing verify that your patch doesn't break muxer_mpeg;
> sometimes I've seen it broken but seemingly harmless code.

Sure, I'll check and be certain.

...and I haven't found any problems yet.

I don't know which patch you refer to, but neither of them should effect
muxer_mpeg directly. The synchronization of the streams being fed to
each muxer may be corrected, but that shouldn't change the muxing itself
unless the muxer is specifically altered to do so. For instance, I alter
muxer_avi to apply stream delays to dwStart. None of the other muxers
recognize dwStart or stream delays at all, and I don't alter them to do
so (though it would be good, later).

-Corey




More information about the MPlayer-dev-eng mailing list