Commit 8df8f52e authored by Ryan C. Gordon's avatar Ryan C. Gordon

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.

--HG--
branch : SDL-1.2
extra : convert_revision : svn%3Ac70aab31-4412-0410-b14c-859654838e24/branches/SDL-1.2%404076
parent 41f37ecc
...@@ -305,17 +305,16 @@ static __inline__ void swizzle_alsa_channels(_THIS) ...@@ -305,17 +305,16 @@ static __inline__ void swizzle_alsa_channels(_THIS)
static void ALSA_PlayAudio(_THIS) static void ALSA_PlayAudio(_THIS)
{ {
int status; int status;
snd_pcm_uframes_t frames_left; snd_pcm_uframes_t samps_left;
const Uint8 *sample_buf = (const Uint8 *) mixbuf; const Uint8 *sample_buf = (const Uint8 *) mixbuf;
const int frame_size = ( ((int) this->spec.channels) * const int sample_size = ((int) (this->spec.format & 0xFF)) / 8;
(((int) (this->spec.format & 0xFF)) / 8) );
swizzle_alsa_channels(this); 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 ) { while ( samps_left > 0 ) {
status = SDL_NAME(snd_pcm_writei)(pcm_handle, sample_buf, frames_left); status = SDL_NAME(snd_pcm_writei)(pcm_handle, sample_buf, samps_left);
if ( status < 0 ) { if ( status < 0 ) {
if ( status == -EAGAIN ) { if ( status == -EAGAIN ) {
SDL_Delay(1); SDL_Delay(1);
...@@ -337,8 +336,8 @@ static void ALSA_PlayAudio(_THIS) ...@@ -337,8 +336,8 @@ static void ALSA_PlayAudio(_THIS)
} }
continue; continue;
} }
sample_buf += status * frame_size; sample_buf += status * sample_size;
frames_left -= status; samps_left -= status;
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment