[Speex-dev] Re: Fix for lsp.c for 16-bit platforms (TI C55x DSP)

[Jean-Marc Valin]

Thanks Jim. Patch applied to svn. Should have been more careful when
merging the original patch.

	Jean-Marc


On Mon, 2006-07-24 at 13:03 -0400, Jim Crichton wrote:
> Jean-Marc,
>  
> Last week I tried the SVN code (build 11700) on the TI C55x DSP, and
> found that operation was broken again.  I traced this to build 11522,
> committed on 5 June.  The problem is in lsp.c, function lsp_to_lpc().
> The line (lsp.c line 461 in build 11700):
>  
>     xin = 1<<(QIMP-1); /* 0.5 in QIMP format */
> 
> evaluates to zero.  The following change corrects the problem:
>  
>     xin = SHL32(EXTEND32(1), (QIMP-1)); /* 0.5 in QIMP format */
> 
> Regards,
>  
> Jim Crichton
>  
>  
>