mrw wrote: > Well, I did not determine anything regarding minimum output buffer > headroom that I could feel confident in. So I will propose adoption of > the same approach as Triode has already taken, which is to limit the > threshold to 1 second. This is working on the very few "crazy high > sample rate" files that I have. PR on its way for your consideration. > > The existing audio output buffer size will only accommodate about 1.1x > seconds at 384k. I saw references to 768k somewhere...
The default maximum sample rate in portaudio for squeezeplay is 384k so my builds won't playback anything higher. The only references to 768k I can think of are in the patch I apply to portaudio for squeezelite mac and windows only. It allows the playback of DSD512 via dop on those two platforms which I've had several requests recently. 1.1x seconds is just squeaking by the minimum required. Perhaps it's time to increase DECODE_AUDIO_BUFFER_SIZE to allow for a few more seconds of 384k? Let's see what the results are with the new 7.8.0r1291 builds which include your PR#9. Thank you for submitting the fix. Ralphy *1*-Touch, *5*-Classics, *3*-Booms, *1*-UE Radio 'Squeezebox client builds' (https://sourceforge.net/projects/lmsclients/files/) 'donations' (https://www.paypal.com/cgi-bin/webscr?cmd=_donations&business=LL5P6365KQEXN&lc=CA&item_name=Squeezebox%20client%20builds¤cy_code=USD&bn=PP%2dDonationsBF%3abtn_donate_SM%2egif%3aNonHosted) always appreciated. ------------------------------------------------------------------------ ralphy's Profile: http://forums.slimdevices.com/member.php?userid=3484 View this thread: http://forums.slimdevices.com/showthread.php?t=113234 _______________________________________________ Squeezecenter mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/squeezecenter
