kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-03 11:25:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

utils.h: Add rounding to float conversion to int.

Browse Source 
Quote from an audio engineer NR9V:
There is a minor issue of a small amount of crossover distortion though as a result of `ast_slinear_saturated_multiply_float()` not rounding the float. This could result in some quiet but potentially audible distortion artifacts in lower volume parts of the signal. If you have for example a sign wave function with a max amplitude of just a few samples, all samples between -1 and 1 will be truncated to zero, resulting in the waveform no longer being a sine wave and in harmonic distortion.

Resolves: #1176
This commit is contained in:
mkmer
2025-03-23 21:04:50 -04:00
parent bf9a6d80f8
commit 6d06d1765f
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
include/asterisk/utils.h
View File

@@ -495,8 +495,11 @@ static force_inline void ast_slinear_saturated_multiply_float(short *input, floa
*input = 32767; *input = 32767;
else if (res < -32768) else if (res < -32768)
*input = -32768; *input = -32768;
else if (res > 0)
*input = (short) (res + 0.5);
else else
*input = (short) res; *input = (short) (res - 0.5);
} }
static force_inline void ast_slinear_saturated_divide(short *input, short *value) static force_inline void ast_slinear_saturated_divide(short *input, short *value)
@@ -511,8 +514,11 @@ static force_inline void ast_slinear_saturated_divide_float(short *input, float
*input = 32767; *input = 32767;
else if (res < -32768) else if (res < -32768)
*input = -32768; *input = -32768;
else if (res > 0)
*input = (short) (res + 0.5);
else else
*input = (short) res; *input = (short) (res - 0.5);
} }
#ifdef localtime_r #ifdef localtime_r