Re: [AD] new ALSA driver

[Chris Robinson <chris.kcat@xxxxxxxxxx>]

On Tuesday 29 August 2006 02:49, Milan Mimica wrote:
> Reuploaded everything to http://www.riteh.hr/~mmimica/allegro-alsa.zip
> The example app now can use streams now.
> The al_audio_set_* functions don't take effect until driver is reopened.

Don't forget, when _al_voice_update returns NULL, you should play silence. You 
currently just rest and return. And I think 24-bit audio actually uses 32-bit 
values, but leaves the top 8 bits unused (so depth/8 won't be right). I'm not 
positive about that, but that's how Allegro is. You also seem to have a lot 
of extrenuous mutex locks.

I've updated the code (attached; strangesoft's ftp seems broken, atm) using 
the latest sound code I have here, added error.c/error.h for the current 
error API, added openal.c, and supplied a simple default.cbd script. I also 
fixed up alsa.c a little (mainly as it related to the threaded loop). I still 
see things I want to clean up, but it doesn't seem to be far off from being 
able to use multiple hardware voices, assuming it just needs more 
snd_pcm_open calls.

Attachment: audio_api.tar.bz2
Description: application/tbz