[MPlayer-dev-eng] [PATCH] Include buffered decoded audio in av sync calculations

Corey Hickey bugfood-ml at fatooh.org
Fri Apr 14 07:43:14 CEST 2006


Corey Hickey wrote:
> Uoti Urpala wrote:
>> On Mon, 2006-04-10 at 16:41 +0300, Uoti Urpala wrote:
>>> typically cause a varying error of about 20 ms. Attached patch fixes
>>> this.
>> This patch also needs *playback_speed, fixed version attached...
> 
> Weren't there some people with variable A-V sync problems and alsa a 
> while ago? I wonder if this patch helps them.

Well, I've tested this and ...I don't see any difference. But, I never 
had this problem before. It would be nice if somebody else could say 
this fixes a problem for them, but, in the absence of that, I'm going to 
figure you know your way around the A-V sync code well enough and give 
an ultimatum:

If nobody speaks up against this patch, I'm going to apply it Sunday night.

I'm a little bit concerned about the ?: test removed in the first hunk, 
though, so if anybody knows why it's there I'd appreciate if they speak up.

-Corey




More information about the MPlayer-dev-eng mailing list