You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2022/06/26 20:12:21 UTC

[commons-numbers] branch master updated: Correct erasure of unnecessary parentheses

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


The following commit(s) were added to refs/heads/master by this push:
     new f31cf16c Correct erasure of unnecessary parentheses
f31cf16c is described below

commit f31cf16cff3d7462ca4bad3c0240ba5935d94785
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Sun Jun 26 21:11:32 2022 +0100

    Correct erasure of unnecessary parentheses
---
 .../main/java/org/apache/commons/numbers/fraction/BigFraction.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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 927b18ba..43ecf80b 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
@@ -1273,8 +1273,8 @@ public final class BigFraction
         BigInteger result = value.shiftRight(bits);
         if (value.testBit(bits - 1) &&
             (hasFractionalBits ||
-             value.getLowestSetBit() < bits - 1) ||
-             value.testBit(bits)) {
+             value.getLowestSetBit() < bits - 1 ||
+             value.testBit(bits))) {
             result = result.add(BigInteger.ONE); //round up
         }