[pulseaudio-discuss] [PATCH 2/3] resampler: Add optional soxr resampler

[Alexander E. Patrakov]

04.08.2014 18:40, Peter Meerwald wrote:
> see http://sourceforge.net/p/soxr/wiki/Home/
>
> Signed-off-by: Peter Meerwald <pmeerw at pmeerw.net>
> Signed-off-by: poljar (Damir Jelić) <poljarinho at gmail.com>
>
> --
>
> rebased and leftover handling is external to _resample() now;
> this fixes a bug in the leftover_length computation since soxr supports
> s16 and float32, hence sizeof(float) is wrong
> ---
>   configure.ac                   |  17 ++++++
>   src/Makefile.am                |   6 ++
>   src/pulsecore/resampler.c      |  14 ++++-
>   src/pulsecore/resampler.h      |   2 +
>   src/pulsecore/resampler/soxr.c | 124 +++++++++++++++++++++++++++++++++++++++++
>   5 files changed, 162 insertions(+), 1 deletion(-)

I'd rather not accept this resampler. It is worse than speex-float-1, 
and, for 44100 -> 48000 Hz resampling, produces audible (according to 
the model) distortions for every full-scale sine wave above 3 kHz.

-- 
Alexander E. Patrakov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: soxr_audibility.png
Type: image/png
Size: 41671 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20140824/defcaf8f/attachment-0002.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: speex-float-1_audibility.png
Type: image/png
Size: 51221 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20140824/defcaf8f/attachment-0003.png>