[FFmpeg-devel] [PATCH] resample: clarify supported resampling.
michaelni at gmx.at
Thu Jun 9 01:43:05 CEST 2011
On Wed, Jun 08, 2011 at 11:48:03AM +0200, Clément Bœsch wrote:
> The attached patch should clarify a bit the audio resampling support.
> Please check if I didn't forget any support; it seems the current error
> message omit for example the downmix from 6 to 2 channels.
> Additionally, the patch fix a sigsegv when requesting 2 channels output
> with 8 channels input.
> Just a question: all the downmix could be supported with a generic code
> somewhat easily; why isn't it done? Is there any special reason?
> Also, the resampling API seems inconsistent (and not very usable for a
> user), can anyone enlighten me on what is planed on that matter?
> : http://www-mmsp.ece.mcgill.ca/documents/audioformats/wave/Samples/Microsoft/8_Channel_ID.wav
> Clément B.
> resample.c | 29 ++++++++++++++++++++++++-----
> 1 file changed, 24 insertions(+), 5 deletions(-)
> 94b7fa4841c258541672b2120f089bd2f6c0a3a2 0001-resample-clarify-supported-resampling.patch
> From b141cc99e951e0a4e8fba11b9ce3f9002ea20f80 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= <clement.boesch at smartjog.com>
> Date: Wed, 8 Jun 2011 11:26:43 +0200
> Subject: [PATCH] resample: clarify supported resampling.
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> This also fix a crash on 8ch → 2ch requested resampling.
should be ok if tested
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel