commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilles Sadowski <gil...@harfang.homelinux.org>
Subject Re: [Math] New warnings from "FindBugs"
Date Thu, 01 Mar 2012 14:33:59 GMT
Hello.

> 
> The version upgrade of the FindBugs plugin led to new discoveries some of
> which are potentially serious bugs:
> 
> * org.apache.commons.math3.ode.nonstiff.GraggBulirschStoerIntegrator: Was
>   method "setStepsizeControl" (note the spelling) intended to override
>   "setStepSizeControl" defined in the parent class?
> 
> * org.apache.commons.math3.linear.SymmLQ: Yet another problem with a
>   probably unnecessary "Serializable"...
> 
> * org.apache.commons.math3.genetics.Chromosome: At line 31 (and 43, where
>   FindBugs warns about strict floating point comparison), is the value
>   intended to be the most negative one, instead of "Double.MIN_VALUE" (which
>   is positive)?

I took care of that one.

> 
> * org.apache.commons.math3.random.EmpiricalDistribution (lines 213, 221,
>   242, 246) and  org.apache.commons.math3.random.ValueServer (line 270):
>   From "FindBugs":
>    Dm: Reliance on default encoding (DM_DEFAULT_ENCODING)
> 
>    Found a call to a method which will perform a byte to String (or String to
>    byte) conversion, and will assume that the default platform encoding is
>    suitable. This will cause the application behaviour to vary between
>    platforms. Use an alternative API and specify a charset name or Charset
>    object explicitly. 
> 
> 
> I think that the last one could be fixed later (unless someone knows how to
> solve it). But for the others, please have look ASAP.
> 

Gilles

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message