On Tue, Jan 27, 2009 at 02:52:29PM -0800, Jeremy Evans wrote:
> Here's an update to the audio/aqualung port.  It adds:
> 
> * sndio backend based on the upstream support I wrote (with a minor
> change from bufsz to appbufsz)
> * wavpack support, since it got added to the ports tree recently
> * use of SUBST_CMD
> 
> There should be a new release of aqualung next month with some cool new
> features (e.g. Lua-based programmable title formatting), but I'm not
> sure if it will make it before the ports lock, so I'm submitting this
> now just in case.  Plus, hopefully if I did something wrong with the
> sndio support, you guys can correct me so I can fix it upstream before
> the next release.

looks good.  the only thing might be instead of

        sndio_par.appbufsz = sndio_par.rate/4;

to request a specific block size and block count like

        sndio_par.round = sndio_par.rate/8;
        sndio_par.appbufsz = sndio_par.round*2;

using 2 blocks per buffer is the most efficient use of the buffer.

-- 
[email protected]
SDF Public Access UNIX System - http://sdf.lonestar.org

Reply via email to