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: svn commit: r1034457 - in /commons/proper/math/branches/MATH_2_X/src: main/java/org/apache/commons/math/ main/java/org/apache/commons/math/exception/ main/java/org/apache/commons/math/linear/ main/java/org/apache/commons/math/ode/ site/xdoc/
Date Fri, 12 Nov 2010 20:58:53 GMT
Hi.

> >>> MATH-440
> >>> New exception "MathUserException" supersedes "FunctionEvaluationException",
> >>> "MatrixVisitorException" and "DerivativeException".
> >>>
> >>> Added:
> >>>     commons/proper/math/branches/MATH_2_X/src/main/java/org/apache/commons/math/exception/MathUserException.java
  (with props)
> >>
> >> Fine, thanks.
> >> I'll change the constructors to replace the String with Localizable as
> >> per the new localization scheme (which is already in place in 2.X branch).
> > 
> > Do you think that it is necessary, given that as a user-level exception,
> > there won't probably exist an enum element corresponding to the error?
> 
> Some users already do have their own implementation of the Localizable
> interface, with their own enum. We do have this in Orekit for example.
> So users are not restricted to the [math] enums.

Hence, I would propose to create a new "MathRuntimeException" (in package
"exception") and move all the code currently in "MathIllegalArgumentException"
over to that class.
Then, "MathIllegalArgumentException" as well as "MathUserException" can
inherit from it.

What do you think?

> [...]


Gilles

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


Mime
View raw message