You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by er...@apache.org on 2019/11/06 15:09:35 UTC
[commons-numbers] 02/04: Formatting nits.
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 <gi...@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;