commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephen Colebourne" <>
Subject Re: [lang] [math] org.apache.commons.lang.math.Fraction class
Date Wed, 02 Jun 2004 23:37:17 GMT
I had a 5 second look yesterday, and realised how big the change is. My
immediate concern is that the Fraction class has lost its final status,
making it less immutable.

I was wondering whether it would be possible to have just one class (rather
than an additional BadFraction subclass) that held an additional field only
used when using non-reduced fractions. Effectively merging the two classes
into one?

Also, I wasn't sure what 'relatively prime' actually meant. Its probably too
mathematical for the target audience in [lang].

Finally, I believe that [lang] is the right place for these (including
BigFraction). They represent missing parts of the JDK, so [lang] is


----- Original Message -----
From: "C. Scott Ananian" <>
> > > What about getNumerator()/getDenominator() - won't these change if 2/4
> > > held differently? I believe that we need to preserve the 2/4 concept
> > > distinct from 1/2, at least in the [lang] version.
> All these concerns have been addressed in the latest version of my patch.
> I'd appreciate review and comments.  Thanks!
>   --scott
> overthrow Echelon justice plastique ASW Yeltsin Sigint assassination
> early warning Soviet  Bush Cocaine terrorist Legion of Doom ammunition
>                          ( )

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

View raw message