# HG changeset patch
# User Ryan C. Gordon <icculus@icculus.org>
# Date 1255422448 0
# Node ID 9716da3531041447d8905572e181ea84853c9178
# Parent  4abf24b03b1dd17d50823b77a64af607e2fb176c
Corrected my snd_pcm_writei fix.

Apparently ALSA says "frames" but it means samples...you don't split this into
 groups by number of channels.

The adventure continues.

diff -r 4abf24b03b1d -r 9716da353104 src/audio/alsa/SDL_alsa_audio.c
--- a/src/audio/alsa/SDL_alsa_audio.c	Tue Oct 13 06:45:02 2009 +0000
+++ b/src/audio/alsa/SDL_alsa_audio.c	Tue Oct 13 08:27:28 2009 +0000
@@ -305,17 +305,16 @@
 static void ALSA_PlayAudio(_THIS)
 {
 	int status;
-	snd_pcm_uframes_t frames_left;
+	snd_pcm_uframes_t samps_left;
 	const Uint8 *sample_buf = (const Uint8 *) mixbuf;
-	const int frame_size = ( ((int) this->spec.channels) *
-	                         (((int) (this->spec.format & 0xFF)) / 8) );
+	const int sample_size = ((int) (this->spec.format & 0xFF)) / 8;
 
 	swizzle_alsa_channels(this);
 
-	frames_left = ((snd_pcm_uframes_t) this->spec.samples) / this->spec.channels;
+	samps_left = ((snd_pcm_uframes_t) this->spec.samples);
 
-	while ( frames_left > 0 ) {
-		status = SDL_NAME(snd_pcm_writei)(pcm_handle, sample_buf, frames_left);
+	while ( samps_left > 0 ) {
+		status = SDL_NAME(snd_pcm_writei)(pcm_handle, sample_buf, samps_left);
 		if ( status < 0 ) {
 			if ( status == -EAGAIN ) {
 				SDL_Delay(1);
@@ -337,8 +336,8 @@
 			}
 			continue;
 		}
-		sample_buf += status * frame_size;
-		frames_left -= status;
+		sample_buf += status * sample_size;
+		samps_left -= status;
 	}
 }