commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Luc Maisonobe <>
Subject Re: svn commit: r1169437 - /commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/
Date Sun, 11 Sep 2011 14:53:28 GMT
Le 11/09/2011 15:51, sebb a écrit :
> On 11 September 2011 14:28, Gilles Sadowski
> <>  wrote:
>> Hi.
>>> + * Precomputed literal arrays are provided in this class to speed up load
>>> + * time.
>> This is an implementation detail, subject to change in the (hopefully
>> not-so-distant) future.
> Disagree.
>> I would certainly not advertize it in the Javadoc.
> I agree here.

I have advertised it as we did introduce some comments about start up 
time some months ago, after a first Jira issue about this.

Please go ahead fixing these comments so they are most useful to users.

> I would also not advertise the ability to change to on the fly
> computation, as I think this facility should be removed.

It's not really on the fly, Gilles set it up as a compile-time option.

> There is quite a lot of code that relates only to the computation.
> The code should be kept, but in a separate class (may need to add some
> package protected methods to provide access).
> I don't think minimising the class source file size is nearly as
> important as the startup time.
> However, there are things that can be done to make the source file
> easier to read.
> Moving the startup code into a separate class would reduce the size of
> the class somewhat.
> The private classes holding preset arrays can be moved to the end of
> the class, and entries can be packaged several per line to shorten the
> file.

This seems fine to me.


> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message