You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Osamu Ikeuchi (JIRA)" <ji...@apache.org> on 2015/08/20 14:50:46 UTC
[jira] [Updated] (MATH-1261) Overflow checks in Fraction
multiply(int) / divide(int)
[ https://issues.apache.org/jira/browse/MATH-1261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Osamu Ikeuchi updated MATH-1261:
--------------------------------
Description:
The member methods multiply(int) / divide(int) in the class org.apache.commons.math3.fraction.Fraction do not have overflow checks.
{code:java}
return new Fraction(numerator * i, denominator);
{code}
should be
{code:java}
return new Fraction(ArithmeticUtils.mulAndCheck(numerator, i), denominator);
{code}
or, considering the case gcd(i, denominator) > 1,
{code:java}
return multiply(new Fraction(i));
{code}
was:
The member methods multiply(int) / divide(int) in the class org.apache.commons.math3.fraction.Fraction do not have overflow checks.
return new Fraction(numerator * i, denominator);
should be
return new Fraction(ArithmeticUtils.mulAndCheck(numerator, i), denominator);
or, considering the case gcd(i, denominator) > 1,
return multiply(new Fraction(i));
> Overflow checks in Fraction multiply(int) / divide(int)
> -------------------------------------------------------
>
> Key: MATH-1261
> URL: https://issues.apache.org/jira/browse/MATH-1261
> Project: Commons Math
> Issue Type: Bug
> Affects Versions: 3.5
> Reporter: Osamu Ikeuchi
> Priority: Minor
>
> The member methods multiply(int) / divide(int) in the class org.apache.commons.math3.fraction.Fraction do not have overflow checks.
> {code:java}
> return new Fraction(numerator * i, denominator);
> {code}
> should be
> {code:java}
> return new Fraction(ArithmeticUtils.mulAndCheck(numerator, i), denominator);
> {code}
> or, considering the case gcd(i, denominator) > 1,
> {code:java}
> return multiply(new Fraction(i));
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)