From 2c8e8944450fdf521808aeae632361e37973a7c1 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <icculus@icculus.org>
Date: Fri, 1 Sep 2006 05:31:36 +0000
Subject: [PATCH] Mac OS X audio backend now supports int32/float32.

---
 src/audio/macosx/SDL_coreaudio.c | 63 ++++++++++++++++++++++++--------
 1 file changed, 47 insertions(+), 16 deletions(-)

diff --git a/src/audio/macosx/SDL_coreaudio.c b/src/audio/macosx/SDL_coreaudio.c
index 9ae88d8a0..59bfdf7c1 100644
--- a/src/audio/macosx/SDL_coreaudio.c
+++ b/src/audio/macosx/SDL_coreaudio.c
@@ -207,25 +207,56 @@ Core_OpenAudio(_THIS, SDL_AudioSpec * spec)
     Component comp;
     ComponentDescription desc;
     struct AudioUnitInputCallback callback;
-    AudioStreamBasicDescription requestedDesc;
+    AudioStreamBasicDescription desc;
+    SDL_AudioFormat test_format = SDL_FirstAudioFormat(spec->format);
+    int valid_datatype = 0;
 
     /* Setup a AudioStreamBasicDescription with the requested format */
-    requestedDesc.mFormatID = kAudioFormatLinearPCM;
-    requestedDesc.mFormatFlags = kLinearPCMFormatFlagIsPacked;
-    requestedDesc.mChannelsPerFrame = spec->channels;
-    requestedDesc.mSampleRate = spec->freq;
+    memset(&desc, '\0', sizeof (AudioStreamBasicDescription));
+    desc.mFormatID = kAudioFormatLinearPCM;
+    desc.mFormatFlags = kLinearPCMFormatFlagIsPacked;
+    desc.mChannelsPerFrame = spec->channels;
+    desc.mSampleRate = spec->freq;
+    desc.mFramesPerPacket = 1;
+
+    while ((!valid_datatype) && (test_format)) {
+        spec->format = test_format;
+        desc.mFormatFlags = 0;
+        /* Just a list of valid SDL formats, so people don't pass junk here. */
+        switch (test_format) {
+            case AUDIO_U8:
+            case AUDIO_S8:
+            case AUDIO_U16LSB:
+            case AUDIO_S16LSB:
+            case AUDIO_U16MSB:
+            case AUDIO_S16MSB:
+            case AUDIO_S32LSB:
+            case AUDIO_S32MSB:
+            case AUDIO_F32LSB:
+            case AUDIO_F32MSB:
+                valid_datatype = 1;
+                desc.mBitsPerChannel = SDL_AUDIO_BITSIZE(spec->format);
+
+                if (SDL_AUDIO_ISFLOAT(spec->format))
+                    desc.mFormatFlags |= kLinearPCMFormatFlagIsFloat;
+                else if (SDL_AUDIO_ISSIGNED(spec->format))
+                    desc.mFormatFlags |= kLinearPCMFormatFlagIsSignedInteger;
+
+                if (SDL_AUDIO_ISBIGENDIAN(spec->format))
+                    desc.mFormatFlags |= kLinearPCMFormatFlagIsBigEndian;
+                break;
+        }
+    }
 
-    requestedDesc.mBitsPerChannel = spec->format & 0xFF;
-    if (spec->format & 0x8000)
-        requestedDesc.mFormatFlags |= kLinearPCMFormatFlagIsSignedInteger;
-    if (spec->format & 0x1000)
-        requestedDesc.mFormatFlags |= kLinearPCMFormatFlagIsBigEndian;
+    if (!valid_datatype) {  /* shouldn't happen, but just in case... */
+        SDL_SetError("Unsupported audio format");
+        return (-1);
+    }
 
-    requestedDesc.mFramesPerPacket = 1;
-    requestedDesc.mBytesPerFrame =
-        requestedDesc.mBitsPerChannel * requestedDesc.mChannelsPerFrame / 8;
-    requestedDesc.mBytesPerPacket =
-        requestedDesc.mBytesPerFrame * requestedDesc.mFramesPerPacket;
+    desc.mBytesPerFrame =
+        desc.mBitsPerChannel * desc.mChannelsPerFrame / 8;
+    desc.mBytesPerPacket =
+        desc.mBytesPerFrame * desc.mFramesPerPacket;
 
 
     /* Locate the default output audio unit */
@@ -252,7 +283,7 @@ Core_OpenAudio(_THIS, SDL_AudioSpec * spec)
                                       kAudioUnitProperty_StreamFormat,
                                       kAudioUnitScope_Input,
                                       0,
-                                      &requestedDesc, sizeof(requestedDesc));
+                                      &desc, sizeof (desc));
     CHECK_RESULT("AudioUnitSetProperty (kAudioUnitProperty_StreamFormat)")
         /* Set the audio callback */
         callback.inputProc = audioCallback;