On Mon, 2014-08-04 at 14:40 +0200, Peter Meerwald wrote: > Hello, > > this patch series splits up the resampler implementation found in pulsecore > into separate files and adds two new resampler implementations: soxr and > libavresample > > the work was mostly done by poljar (Damir Jeli?) during GSoC'13, this is just > a rebase > > my goal would be to establish libavresample as the new default resampler and > drop the ffmpeg code copied into PA currently; don't worry, this would be > further work based on the feedback received :) The split looks good to go in now. For the other two, I think it would be nice to have some measurements/justification. We have the resampler test to do CPU measurements (and I see that poljar already did some comparisons that show soxr to be favourable in the int case at least), but I think we need to also have a justification based on quality. Any thoughts on measuring that? I should also add that for libavresample to be the default implementation, it needs to be easily available in distributions, and I think this might be a concern unless the packaging is split off (at least on my Fedora box, this comes from rpmfusion and not offcial Fedora packages). Cheers, Arun
