This is an automated email from the ASF dual-hosted git repository. aherbert pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-numbers.git
commit 4e0f420890660e945ed7403518ba06ec0ece812b Author: Alex Herbert <[email protected]> AuthorDate: Tue Aug 25 13:16:31 2026 +0100 Checkstyle: Remove unnecessary parentheses --- .../src/main/java/org/apache/commons/numbers/core/Precision.java | 4 ++-- .../main/java/org/apache/commons/numbers/fraction/BigFraction.java | 4 ++-- .../java/org/apache/commons/numbers/quaternion/QuaternionTest.java | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/commons-numbers-core/src/main/java/org/apache/commons/numbers/core/Precision.java b/commons-numbers-core/src/main/java/org/apache/commons/numbers/core/Precision.java index af4712d8..efe80a3a 100644 --- a/commons-numbers-core/src/main/java/org/apache/commons/numbers/core/Precision.java +++ b/commons-numbers-core/src/main/java/org/apache/commons/numbers/core/Precision.java @@ -478,8 +478,8 @@ public final class Precision { int scale, RoundingMode roundingMode) { try { - final double rounded = (new BigDecimal(Double.toString(x)) - .setScale(scale, roundingMode)) + final double rounded = new BigDecimal(Double.toString(x)) + .setScale(scale, roundingMode) .doubleValue(); // MATH-1089: negative values rounded to zero should result in negative zero return rounded == POSITIVE_ZERO ? POSITIVE_ZERO * x : rounded; diff --git a/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/BigFraction.java b/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/BigFraction.java index 611d868f..d69c6bbc 100644 --- a/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/BigFraction.java +++ b/commons-numbers-fraction/src/main/java/org/apache/commons/numbers/fraction/BigFraction.java @@ -768,7 +768,7 @@ public final class BigFraction num = numerator.add(value.numerator); den = denominator; } else { - num = (numerator.multiply(value.denominator)).add((value.numerator).multiply(denominator)); + num = numerator.multiply(value.denominator).add(value.numerator.multiply(denominator)); den = denominator.multiply(value.denominator); } @@ -841,7 +841,7 @@ public final class BigFraction num = numerator.subtract(value.numerator); den = denominator; } else { - num = (numerator.multiply(value.denominator)).subtract((value.numerator).multiply(denominator)); + num = numerator.multiply(value.denominator).subtract(value.numerator.multiply(denominator)); den = denominator.multiply(value.denominator); } diff --git a/commons-numbers-quaternion/src/test/java/org/apache/commons/numbers/quaternion/QuaternionTest.java b/commons-numbers-quaternion/src/test/java/org/apache/commons/numbers/quaternion/QuaternionTest.java index 6b3810a7..d33e4c16 100644 --- a/commons-numbers-quaternion/src/test/java/org/apache/commons/numbers/quaternion/QuaternionTest.java +++ b/commons-numbers-quaternion/src/test/java/org/apache/commons/numbers/quaternion/QuaternionTest.java @@ -136,7 +136,7 @@ class QuaternionTest { // Conj(qA * qB) = Conj(qB) * Conj(qA) final Quaternion productOfConjugate = qB.conjugate().multiply(qA.conjugate()); - final Quaternion conjugateOfProduct = (qA.multiply(qB)).conjugate(); + final Quaternion conjugateOfProduct = qA.multiply(qB).conjugate(); Assertions.assertEquals(productOfConjugate.getW(), conjugateOfProduct.getW(), EPS); Assertions.assertEquals(productOfConjugate.getX(), conjugateOfProduct.getX(), EPS);
