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 2025/03/26 11:53:18 UTC

(commons-numbers) 11/14: Sonar fix: remove useless assignment

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 aeed8e10716c468e31f32f9ca445329065e23042
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Tue Mar 25 15:14:48 2025 +0000

    Sonar fix: remove useless assignment
---
 .../apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java   | 1 -
 1 file changed, 1 deletion(-)

diff --git a/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java b/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
index 2daf549a..d44c306d 100644
--- a/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
+++ b/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
@@ -902,7 +902,6 @@ class CommonsLangPortedFractionTest {
 
         // Should overflow
         assertOperationThrows(ArithmeticException.class, Fraction.of(1, Integer.MAX_VALUE), Fraction.of(1, Integer.MAX_VALUE - 1), Fraction::subtract);
-        f = f1.subtract(f2);
 
         // denominator should not be a multiple of 2 or 3 to trigger overflow
         assertOperationThrows(ArithmeticException.class, Fraction.of(Integer.MIN_VALUE, 5), Fraction.of(1, 5), Fraction::subtract);