[Mitgcm-support] Re: fftpack.F (PR#17)

[mitgcm-support at dev.mitgcm.org]

Comments

1. #ifdef ALLOW_
  as a complete wrapper for code leaves blank files.
  Some compilers object to this.
  Thats why .genmakerc
   DISABLE = ( ... zonalfilt ... ) 
  is a useful option.
  You could add a dummy routine to fftpack.F though.

2. At some stage we will want to use an FFT on the Sun so 
   then I guess it will get fixed?

3. Is zonalfilt designed to allow any othe FFT library?

4. How come zonalfilt isn't disabled by default.

adcroft at mit.edu wrote:
> 
> fftpack.F in pkg/zonal_filt/ doesn't go through the Sun compiler.
> 
> Chris, do you still object to using #ifdef ALLOW_  to wrap the code?