psVectorFFT/psImageFFT direction parameter should be bool

[robert.desonia@…]

I suggest that the direction parameter of psVectorFFT/psImageFFT be changed to
bool instead of int as only two options really exist: forward or reverse.

Could also change the parameter name to 'isForward' to make it easy to figure
out that TRUE is forward.

-rdd

[robert.desonia@…]

Given the issue of communicating if a real-result is desired for an IFFT or not,
I suggest that the direction parameter be converted to a bit flag with the
following enum driving it:

typedef enum {

PS_FFT_FORWARD = 0, /< psImageFFT/psVectorFFT should

perform a forward FFT.

PS_FFT_REVERSE = 1, /< psImageFFT/psVectorFFT should

perform a reverse FFT.

PS_FFT_REAL_RESULT = 2

/< psImageFFT/psVectorFFT should return a real image.

This is valid for

• only reverse FFT, i.e., the psImageFFT/psVectorFFT

flag parameter

• should appear as PS_FFT_REVERSE+PS_FFT_REAL_RESULT. */

} psFftFlags;

[robert.desonia@…]

Sorry for the poor line-wrapping of the last post.

Basically, a forward FFT would look like:

out = psImageFFT(in,PS_FFT_FORWARD);

a reverse FFT would look like:

out = psImageFFT(in,PS_FFT_REVERSE);

and a reverse FFT with a real resultant would look like:

out = psImageFFT(in,PS_FFT_REVERSE+PS_FFT_REAL_RESULT);
(or, if you prefer bitwise operators)
out = psImageFFT(in,PS_FFT_REVERSE | PS_FFT_REAL_RESULT);

[Paul Price]

Suggested implemented in SDRS.