This is an automated email from the ASF dual-hosted git repository. erans pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-numbers.git
commit 3af49d48c34de899e0110990dae7e5bc3a6554e6 Author: Gilles Sadowski <gil...@harfang.homelinux.org> AuthorDate: Wed Nov 6 16:03:16 2019 +0100 Formatting nits. --- .../java/org/apache/commons/numbers/fraction/Fraction.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/Fraction.java b/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/Fraction.java index 228f236..8101a8d 100644 --- a/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/Fraction.java +++ b/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/Fraction.java @@ -115,17 +115,21 @@ public class Fraction p2 = (a1 * p1) + p0; q2 = (a1 * q1) + q0; - if ((Math.abs(p2) > overflow) || (Math.abs(q2) > overflow)) { + if (Math.abs(p2) > overflow || + Math.abs(q2) > overflow) { // in maxDenominator mode, if the last fraction was very close to the actual value // q2 may overflow in the next iteration; in this case return the last one. - if (epsilon == 0.0 && Math.abs(q1) < maxDenominator) { + if (epsilon == 0.0 && + Math.abs(q1) < maxDenominator) { break; } throw new FractionException(FractionException.ERROR_CONVERSION, value, p2, q2); } double convergent = (double)p2 / (double)q2; - if (n < maxIterations && Math.abs(convergent - value) > epsilon && q2 < maxDenominator) { + if (n < maxIterations && + Math.abs(convergent - value) > epsilon && + q2 < maxDenominator) { p0 = p1; p1 = p2; q0 = q1;