[pulseaudio-discuss] [PATCH] make speex optional, make Adrian AEC optional
Arun Raghavan
arun.raghavan at collabora.co.uk
Thu Dec 8 02:15:12 PST 2011
Hi,
On Wed, 2011-12-07 at 15:20 +0100, Peter Meerwald wrote:
> Hello,
>
> attached patch makes
> - the speex library optional (i.e. the speex resampler and the speex
> acoustic echo canceller, AEC)
> - the Adrian AEC implementation in the module-echo-cancel optional
>
> reducing dependencies and unneeded code is desirable in embedded scenarios
>
> note that the patch changes the default resampler in pa_resampler_new() in
> case speex is not available, so
> speex -> libsamplerate -> ffmpeg
>
> please comments
Looks good in principle. Some minor niggles. :)
1. This should be split into separate patches for adrian and speex
(trivial, I can just do this while merging the patch)
2. I'd rather pick ffmpeg as the default if speex isn't selected. The
libsamplerate resampling, while theoretically higher quality, is quite
CPU-intensive.
3. We should check to make sure that at least one canceller is selected
at configure time
Possibly we should just drop the adian canceller altogether. I don't
think it has any practical purpose given that it's nowhere near the
other two in quality.
Regards,
Arun
More information about the pulseaudio-discuss
mailing list