[MPlayer-dev-eng] [fix] ao_alsa: playing high-res audio

[Uoti Urpala]

On Thu, 2009-08-20 at 19:58 +0200, Clemens Ladisch wrote:
> Shane W wrote:
> > For these streams, I needed toenable:
> > #define BUFFERTIME 1
> > in ao_alsa.c which expanded the buffer sizes appropriately.
> > # Is there any reason this isn't on by default?
> 
> Years ago, I had heard rumors that the A/V sync code might get into
> problems if the audio output wants to prefetch too much audio data,
> so I did not touch that code.  However, this is probably not true
> anymore, if it ever was.

The old audio playback (not so much sync) code could have had problems
in that case, but I fixed those sometime in the last couple of years.

> In any case, using a constant buffer time, instead of buffer length in
> frames, should be safe.
> 
> I will rewrite the buffer parameters selection code in the next few days.

Yes using constant time is better.