Comments about FFT PATCH ready



On Sep 8, 2009, at 7:08 PM, Raymond Toy wrote:

> ?iga Lenar?i? wrote:
>>
>>
>> It works nicely with 'flonum', yes. I changed it to use add and mul,
>> added your changes from CVS, used flonum instead of double-float...
>> See the attached file.
>>
> These changes have been committed, more or less.  I made a few small
> changes to match Lisp style better, and to make sure fft-dif-internal
> gets the correct initial values for the general flonum type.
>
> I ran rtest_fft, and all tests pass, whatever that means.
>
> Ray
>

Great! You even included the 'defsystem' part for compilation, very  
nice. In the future, other fft reliant transformations can be added  
(sine transform, cosine transform, perhaps a handy power spectrum  
estimation function, that Woolet uses in his lectures, since it's the  
most common fft usage..).


Thanks,
Ziga