[pulseaudio-discuss] regression in pacat

[Colin Guthrie]

'Twas brillig, and Lennart Poettering at 17/02/10 02:36 did gyre and gimble:
> On Thu, 11.02.10 23:34, Colin Guthrie (gmane at colin.guthr.ie) wrote:
> 
>>
>> 'Twas brillig, and Colin Guthrie at 11/02/10 22:52 did gyre and gimble:
>>> 'Twas brillig, and pl bossart at 11/02/10 21:41 did gyre and gimble:
>>>> Hi Lennart, hope you had a good time in India.
>>>> paplay seems to be broken in git master, git bisect tells me the culprit is:
>>>>
>>>> fb55798a3eb73b4c00225232408b766f74533409 is first bad commit
>>>> commit fb55798a3eb73b4c00225232408b766f74533409
>>>> Author: Lennart Poettering <lennart at poettering.net>
>>>> Date:   Fri Jan 15 01:25:21 2010 +0100
>>>>
>>>>     pacat: allow configuration of latency in msec
>>>>
>>>> With this commit paplay is stuck, there's no audio out and no update
>>>> of time/latency.
>>>
>>> Ahh so that's where the problem lies... I'll try and fix if Lennart
>>> doesn't beat me to it.
>>
>> This fixes it for me:
>> ff2091b pacat: Don't use any buffer attr if we don't set any
>> latency/process time params
>>
>> May not be the nicest fix in the world but it'll do until Lennart can
>> review properly :)
> 
> Actually I had I really wanted to write what I wrote. i.e. passing
> NULL as buffer_attr means "use the default latency" (which is
> 250ms). Passing a buffer_attr with all values set to -1 means "i don't
> care about latency" (which ideally means 2s latency).
> 
> So, yes, I actually meant what I did in fb55798a... Now the question
> is why this doesn't work for Pierre.

For clarity (not sure if I mentioned it) but I did experience the same
problem too, so it wasn't just Pierre that was affected.

To be fair tho', I did only test it with small samples - e.g. less than
2s, so perhaps the problem is that the buffer simply never gets full
enough to start playing?

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mandriva Linux Contributor [http://www.mandriva.com/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]