Fixed silly logic thing.
authorRyan C. Gordon <icculus@icculus.org>
Thu, 31 Aug 2006 22:22:34 +0000
changeset 1998 cbac0f77a799
parent 1997 46319c67b3d7
child 1999 dd4b184b3050
Fixed silly logic thing.
src/audio/amigaos/SDL_ahiaudio.c
src/audio/baudio/SDL_beaudio.cc
--- a/src/audio/amigaos/SDL_ahiaudio.c	Thu Aug 31 22:15:07 2006 +0000
+++ b/src/audio/amigaos/SDL_ahiaudio.c	Thu Aug 31 22:22:34 2006 +0000
@@ -227,16 +227,14 @@
 {
 //      int width;
     SDL_AudioFormat test_format = SDL_FirstAudioFormat(spec->format);
-    int valid_datatype = 0;
 
     D(bug("AHI opening...\n"));
 
     /* Determine the audio parameters from the AudioSpec */
-    while ((!valid_datatype) && (test_format)) {
+    while (test_format) {
         switch (test_format) {
             case AUDIO_S8:
                 D(bug("AUDIO_S8...\n"));
-                valid_datatype = 1;
                 spec->format = AUDIO_S8;
                 this->hidden->bytespersample = 1;
                 if (spec->channels < 2)
@@ -247,7 +245,6 @@
 
             case AUDIO_S16MSB:
                 D(bug("AUDIO_S16MSB...\n"));
-                valid_datatype = 1;
                 spec->format = AUDIO_S16MSB;
                 this->hidden->bytespersample = 2;
                 if (spec->channels < 2)
@@ -258,7 +255,6 @@
 
             case AUDIO_S32MSB:
                 D(bug("AUDIO_S32MSB...\n"));
-                valid_datatype = 1;
                 spec->format = AUDIO_S32MSB;
                 this->hidden->bytespersample = 4;
                 if (spec->channels < 2)
@@ -273,7 +269,7 @@
         }
     }
 
-    if (!valid_datatype) { /* shouldn't happen, but just in case... */
+    if (!test_format) { /* shouldn't happen, but just in case... */
         SDL_SetError("Unsupported audio format");
         return (-1);
     }
--- a/src/audio/baudio/SDL_beaudio.cc	Thu Aug 31 22:15:07 2006 +0000
+++ b/src/audio/baudio/SDL_beaudio.cc	Thu Aug 31 22:22:34 2006 +0000
@@ -157,55 +157,46 @@
     {
         media_raw_audio_format format;
         SDL_AudioFormat test_format = SDL_FirstAudioFormat(spec->format);
-        int valid_datatype = 0;
 
         /* Parse the audio format and fill the Be raw audio format */
         memset(&format, '\0', sizeof (media_raw_audio_format));
         format.byte_order = B_MEDIA_LITTLE_ENDIAN;
         format.frame_rate = (float) spec->freq;
         format.channel_count = spec->channels;  /* !!! FIXME: support > 2? */
-        while ((!valid_datatype) && (test_format)) {
+        while (test_format) {
             spec->format = test_format;
             switch (test_format) {
                 case AUDIO_S8:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_CHAR;
                     break;
 
                 case AUDIO_U8:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_UCHAR;
                     break;
 
                 case AUDIO_S16LSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_SHORT;
                     break;
 
                 case AUDIO_S16MSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_SHORT;
                     format.byte_order = B_MEDIA_BIG_ENDIAN;
                     break;
 
                 case AUDIO_S32LSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_INT;
                     break;
 
                 case AUDIO_S32MSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_INT;
                     format.byte_order = B_MEDIA_BIG_ENDIAN;
                     break;
 
                 case AUDIO_F32LSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_FLOAT;
                     break;
 
                 case AUDIO_F32MSB:
-                    valid_datatype = 1;
                     format.format = media_raw_audio_format::B_AUDIO_FLOAT;
                     format.byte_order = B_MEDIA_BIG_ENDIAN;
                     break;
@@ -218,7 +209,7 @@
 
         format.buffer_size = spec->samples;
 
-        if (!valid_datatype) { /* shouldn't happen, but just in case... */
+        if (!test_format) { /* shouldn't happen, but just in case... */
             SDL_SetError("Unsupported audio format");
             return (-1);
         }