Looks ok to me.

The only other alternative I can think of that may make things easier if anything changes in the future would be to just copy NestableException into the math package hierarchy under something like o.a.c.m.exception.NestableException and extend it from there.

-Mark

Phil Steitz wrote:

I just opened a ticket (#29948) and attached a patch that pulls the implementation code from lang's NestableException into MathException. This would eliminate the [lang] dependency. The full implementation is included, which is probably overkill and may push things over the "small amount of code" limit. Before committing, I wanted to give others the opportunity to suggest alternatives. In any case, I would need to add some more test cases (and test with 1.3, rather than commenting out the 1.4 check as the patch does).

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to