Re: [FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-06 Thread Rob Sykes
On Mon, Mar 06, 2017 at 03:19:48AM +, Rostislav Pehlivanov wrote: > The soxr resampler is slower and worse than the swr resampler, > hence drop the former. With git master of ffmpeg and libsoxr 0.1.1, using default parameters, on x86, soxr is indeed a little slower than swr; however many

Re: [FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-06 Thread Michael Niedermayer
On Mon, Mar 06, 2017 at 03:19:48AM +, Rostislav Pehlivanov wrote: > The soxr resampler is slower and worse than the swr resampler, > hence drop the former. > > Signed-off-by: Rostislav Pehlivanov > --- > Changelog | 1 + > MAINTAINERS

Re: [FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-06 Thread Paul B Mahol
On 3/6/17, Rostislav Pehlivanov wrote: > The soxr resampler is slower and worse than the swr resampler, > hence drop the former. Why you think its worse? There are some popular tests with all resampleres being tested and I can confirm that swr defaults are hardly better than

Re: [FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-05 Thread James Almer
On 3/6/2017 12:19 AM, Rostislav Pehlivanov wrote: > The soxr resampler is slower and worse than the swr resampler, > hence drop the former. > > Signed-off-by: Rostislav Pehlivanov > --- > Changelog | 1 + > MAINTAINERS |

Re: [FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-05 Thread Lou Logan
On Sun, Mar 5, 2017, at 06:19 PM, Rostislav Pehlivanov wrote: > The soxr resampler is slower and worse than the swr resampler, > hence drop the former. Can you explain what you mean by "worse"? Not that I don't trust your judgment, but I'm just curious.

[FFmpeg-devel] [PATCH] lswr: remove support for libsoxr resampling

2017-03-05 Thread Rostislav Pehlivanov
The soxr resampler is slower and worse than the swr resampler, hence drop the former. Signed-off-by: Rostislav Pehlivanov --- Changelog | 1 + MAINTAINERS | 1 - configure | 5 +-