Ronald S. Bultje wrote:
Hi,
On Tue, Nov 10, 2009 at 9:45 AM, Michael Niedermayer <michae...@gmx.at> wrote:
On Tue, Nov 10, 2009 at 05:49:48AM +0100, Vitor Sessak wrote:
vitor wrote:
Author: vitor
Date: Tue Nov 10 05:49:08 2009
New Revision: 5433
Log:
Use the already committed function ff_set_min_dist_lsf().
Note that this changes slightly the output,
which is closer to the correct (that is encoder input)?
Most likely the double one. I don't mind making LSFs/LPCs doubles,
there's very few of them (10-16) per set and ff_acelp_lspd2lpc()
already uses doubles anyway (probably for the same reason). Vitor?
Reynaldo?
The attached patch makes that the output before and after r5433 is
unchanged.
-Vitor
Index: libavcodec/lsp.c
===================================================================
--- libavcodec/lsp.c (revision 20502)
+++ libavcodec/lsp.c (working copy)
@@ -47,7 +47,7 @@
lsfq[lp_order-1] = FFMIN(lsfq[lp_order-1], lsfq_max);//Is warning required ?
}
-void ff_set_min_dist_lsf(float *lsf, float min_spacing, int size)
+void ff_set_min_dist_lsf(float *lsf, double min_spacing, int size)
{
int i;
float prev = 0.0;
Index: libavcodec/lsp.h
===================================================================
--- libavcodec/lsp.h (revision 20502)
+++ libavcodec/lsp.h (working copy)
@@ -50,7 +50,7 @@
* @param min_spacing minimum distance between two consecutive lsf values
* @param size size of the lsf vector
*/
-void ff_set_min_dist_lsf(float *lsf, float min_spacing, int order);
+void ff_set_min_dist_lsf(float *lsf, double min_spacing, int order);
/**
* \brief Convert LSF to LSP
_______________________________________________
FFmpeg-soc mailing list
FFmpeg-soc@mplayerhq.hu
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc