Browse Source

audio: Fix range on float-to-int data clamping.

I can't tell if there was a good reason for this or it was just me getting
numbers wrong due to exhaustion.
Ryan C. Gordon 7 years ago
parent
commit
a07e5815a5
1 changed files with 4 additions and 4 deletions
  1. 4 4
      src/audio/SDL_audiotypecvt.c

+ 4 - 4
src/audio/SDL_audiotypecvt.c

@@ -174,7 +174,7 @@ SDL_Convert_F32_to_S8_Scalar(SDL_AudioCVT *cvt, SDL_AudioFormat format)
         if (sample >= 1.0f) {
             *dst = 127;
         } else if (sample <= -1.0f) {
-            *dst = -127;
+            *dst = -128;
         } else {
             *dst = (Sint8)(sample * 127.0f);
         }
@@ -226,7 +226,7 @@ SDL_Convert_F32_to_S16_Scalar(SDL_AudioCVT *cvt, SDL_AudioFormat format)
         if (sample >= 1.0f) {
             *dst = 32767;
         } else if (sample <= -1.0f) {
-            *dst = -32767;
+            *dst = -32768;
         } else {
             *dst = (Sint16)(sample * 32767.0f);
         }
@@ -250,7 +250,7 @@ SDL_Convert_F32_to_U16_Scalar(SDL_AudioCVT *cvt, SDL_AudioFormat format)
     for (i = cvt->len_cvt / sizeof (float); i; --i, ++src, ++dst) {
         const float sample = *src;
         if (sample >= 1.0f) {
-            *dst = 65534;
+            *dst = 65535;
         } else if (sample <= -1.0f) {
             *dst = 0;
         } else {
@@ -278,7 +278,7 @@ SDL_Convert_F32_to_S32_Scalar(SDL_AudioCVT *cvt, SDL_AudioFormat format)
         if (sample >= 1.0f) {
             *dst = 2147483647;
         } else if (sample <= -1.0f) {
-            *dst = -2147483647;
+            *dst = -2147483648;
         } else {
             *dst = ((Sint32)(sample * 8388607.0f)) << 8;
         }