Speex dev - Mar 2010 - Missing line in preprocess_analysis

Hi,
   In function preprocess_analysis to calculate power spectrum, the last element
of ps is missing.

   /* Power spectrum */
   ps[0]=MULT16_16(st->ft[0],st->ft[0]);
   for (i=1;i<N;i++)
      ps[i]=MULT16_16(st->ft[2*i-1],st->ft[2*i-1]) +
MULT16_16(st->ft[2*i],st->ft[2*i]);

   /*MISSING*/
   ps[N-1] = MULT16_16(st->ft[2*N-1],st->ft[2*N-1]);

   Should it be a bug?
 				

????????QianBin
????????cola_tin at 163.com
??????????2010-03-04

It's voluntarily omitted because it would otherwise end up in ps[N], 
which means I'd have to make ps bigger for no useful reason (who cares 
about that freq anyway).

	Jean-Marc

On 2010-03-04 02:03, QianBin wrote:
> Hi,
>     In function preprocess_analysis to calculate power spectrum, the last
element of ps is missing.
>
>     /* Power spectrum */
>     ps[0]=MULT16_16(st->ft[0],st->ft[0]);
>     for (i=1;i<N;i++)
>        ps[i]=MULT16_16(st->ft[2*i-1],st->ft[2*i-1]) +
MULT16_16(st->ft[2*i],st->ft[2*i]);
>
>     /*MISSING*/
>     ps[N-1] = MULT16_16(st->ft[2*N-1],st->ft[2*N-1]);
>
>     Should it be a bug?
>   				
>
> ????????QianBin
> ????????cola_tin at 163.com
> ??????????2010-03-04
> _______________________________________________
> Speex-dev mailing list
> Speex-dev at xiph.org
> http://lists.xiph.org/mailman/listinfo/speex-dev