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

Reply via email to