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

Rudolf Polzer divVerent at alientrap.org
Mon Feb 21 19:36:20 CET 2011


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,

Rudolf Polzer


More information about the MPlayer-dev-eng mailing list