[MPlayer-dev-eng] [PATCH] Mencoder AV sync

Ivan Kalvachev ikalvachev at gmail.com
Mon Feb 21 23:16:08 CET 2011


On 2/21/11, Rudolf Polzer <divVerent at alientrap.org> wrote:
> On Mon, Feb 21, 2011 at 07:28:21PM +0100, Reimar Döffinger wrote:
>> On Sat, Feb 19, 2011 at 01:56:16PM +0100, Rudolf Polzer wrote:
>> > On Sat, Feb 19, 2011 at 12:36:36AM -0800, Tim Xiao wrote:
>> > > Hi mplayer devs,
>> > >
>> > > I found an AV sync issue with mencoder. For certain audio formats
>> > > (like FLAC), the encoder would mistakenly think that the audio is
>> > > ahead. This causes it to skip a frame every 0.3 seconds or so, and the
>> > > resulting encode would have horrible AV sync issues. After some
>> > > experimenting, I discovered that copying a portion of the mplayer.c AV
>> > > sync code to mencoder.c fixes the problem. While I don't think copying
>> > > code is the best approach, its the best I could come up with. On a
>> > > related note, I couldn't figure out if I was supposed to use 4 spaces,
>> > > 2 spaces or tabs for indentation. So I guessed 2 spaces. :P
>> > >
>> > > Tim Xiao
>> >
>> > This fix was already sent by me before, but did not get applied. See:
>> > http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2010-July/065487.html
>> >
>> > Maybe you won't get ignored like I did.
>>
>> That's not fair, you there were quite a few comments.
>> I did forget about it though, and I am sorry about that.
>
> Yes, but it had been ignored after the comments had been addressed.
>
> But well, there probably is simply lack of a means to track what patches
> have
> been submitted and are in which status.
>
> Best regards,

And how many times I have asked you on IRC to repost that exact patch?
:P

I hope things get moving now.


More information about the MPlayer-dev-eng mailing list