Re: [math] Package transform revisited

2012-02-10 Thread Sébastien Brisard
Dear all, 2012/2/9 Gilles Sadowski : > On Thu, Feb 09, 2012 at 11:02:36AM +0100, Luc Maisonobe wrote: >> Le 09/02/2012 10:50, Sébastien Brisard a écrit : >> > Hi Luc, >> >> >> >> I agree with you, enums are much better. There are other places in >> >> [math] where we use boolean or even ints for s

Re: [math] Package transform revisited

2012-02-09 Thread Gilles Sadowski
On Thu, Feb 09, 2012 at 11:02:36AM +0100, Luc Maisonobe wrote: > Le 09/02/2012 10:50, Sébastien Brisard a écrit : > > Hi Luc, > >> > >> I agree with you, enums are much better. There are other places in > >> [math] where we use boolean or even ints for such things. They mainly > >> came for pre-jav

Re: [math] Package transform revisited

2012-02-09 Thread Luc Maisonobe
Le 09/02/2012 10:50, Sébastien Brisard a écrit : > Hi Luc, >> >> I agree with you, enums are much better. There are other places in >> [math] where we use boolean or even ints for such things. They mainly >> came for pre-java 5 era when enums where not available. >> >> Luc >> > And what do you thin

Re: [math] Package transform revisited

2012-02-09 Thread Sébastien Brisard
Hi Luc, > > I agree with you, enums are much better. There are other places in > [math] where we use boolean or even ints for such things. They mainly > came for pre-java 5 era when enums where not available. > > Luc > And what do you think of replacing - transform(double[]) with transform(double[]

Re: [math] Package transform revisited

2012-02-09 Thread Luc Maisonobe
Le 09/02/2012 08:46, Sébastien Brisard a écrit : > Hi, > while working on MATH-732, I've made some amendments to the > FastFourierTransformer API. Basically, I've removed the factory > methods create() and createUnitary() I had previously introduced in > MATH-677. Instead, I introduced an enum {STA

[math] Package transform revisited

2012-02-08 Thread Sébastien Brisard
Hi, while working on MATH-732, I've made some amendments to the FastFourierTransformer API. Basically, I've removed the factory methods create() and createUnitary() I had previously introduced in MATH-677. Instead, I introduced an enum {STANDARD, UNITARY}. This allows the specification of the type