Use the proper lower bound when doing saturation arithmetic.

16 bit signed integers have a range of [-32768, 32768).  The existing code
was using the interval (-32768, 32768) instead.  This patch fixes that.

Review: https://reviewboard.asterisk.org/r/2479/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@386929 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Sean Bright
2013-04-30 13:45:40 +00:00
parent eeaac15fe9
commit cf90e44d99

View File

@@ -307,8 +307,8 @@ static force_inline void ast_slinear_saturated_add(short *input, short *value)
res = (int) *input + *value;
if (res > 32767)
*input = 32767;
else if (res < -32767)
*input = -32767;
else if (res < -32768)
*input = -32768;
else
*input = (short) res;
}
@@ -320,8 +320,8 @@ static force_inline void ast_slinear_saturated_subtract(short *input, short *val
res = (int) *input - *value;
if (res > 32767)
*input = 32767;
else if (res < -32767)
*input = -32767;
else if (res < -32768)
*input = -32768;
else
*input = (short) res;
}
@@ -333,8 +333,8 @@ static force_inline void ast_slinear_saturated_multiply(short *input, short *val
res = (int) *input * *value;
if (res > 32767)
*input = 32767;
else if (res < -32767)
*input = -32767;
else if (res < -32768)
*input = -32768;
else
*input = (short) res;
}