Comments about FFT PATCH ready



?iga Lenar?i? wrote:
>
> 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
>>
>
> On a side note, fft.usg is rather outdated and it's perhaps
> unnecessary to have user documentation in two places (I would simply
> remove it).
It would be much nicer if maxima knew how to handle documentation
provided in share packages so that they don't have to be a part of the
core maxima documentation.  I'm thinking of how xemacs (and emacs?)
handles info files.  xemacs looks at the info directories and builds up
the info dynamically.  Hence, add-on packages can do whatever they want
and xemacs recognizes them and integrates it into the info reader as if
it were a part of the xemacs distribution.

This would allow the package writers to keep documentation up-to-date
without developers having to spend time integrating said documentation
into maxima itself.

> fft.dem is outdated also - it can be either removed or updated (if
> anybody even looks at these .dem, it's been broken for some time but
> no complaints from the users).
I think the dem files are kind of nice to show off what the package can do.

Ray