fix: use correct var name

lbressler13 · Jan 23, 2024 · aa7a355 · aa7a355
1 parent f02a2c6
commit aa7a355
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/...rs/src/main/kotlin/xyz/lbres/exactnumbers/exactfraction/ExactFractionOverflowException.kt b/...rs/src/main/kotlin/xyz/lbres/exactnumbers/exactfraction/ExactFractionOverflowException.kt
@@ -2,7 +2,7 @@ package xyz.lbres.exactnumbers.exactfraction
 
 /**
  * [ArithmeticException] for ExactFraction overflow.
- * Has field for string of value that caused overflow.
+ * Has field for string representation of value that caused overflow
  */
 class ExactFractionOverflowException() : ArithmeticException() {
     override var message: String? = null

diff --git a/exact-numbers/src/main/kotlin/xyz/lbres/exactnumbers/exactfraction/exactFractionHelpers.kt b/exact-numbers/src/main/kotlin/xyz/lbres/exactnumbers/exactfraction/exactFractionHelpers.kt
@@ -26,13 +26,13 @@ internal fun simplifyFraction(numerator: BigInteger, denominator: BigInteger): T
 
     // move negatives to numerator
     if (newDenominator.isNegative()) {
-        newNumerator = -numerator
-        newDenominator = -denominator
+        newNumerator = -newNumerator
+        newDenominator = -newDenominator
     }
 
     // simplify using greatest common divisor
     if (newNumerator != BigInteger.ZERO) {
-        val gcd = getGCD(numerator, denominator)
+        val gcd = getGCD(newNumerator, newDenominator)
         newNumerator /= gcd
         newDenominator /= gcd
     }

diff --git a/exact-numbers/src/test/kotlin/xyz/lbres/exactnumbers/expressions/term/getComponentsTests.kt b/exact-numbers/src/test/kotlin/xyz/lbres/exactnumbers/expressions/term/getComponentsTests.kt
@@ -17,7 +17,6 @@ private val testNumber1 = TestNumber(ExactFraction(3, 4))
 private val testNumber2 = TestNumber(ExactFraction.SEVEN)
 private val one = ExactFraction.ONE
 
-@Suppress("Deprecated")
 fun runGetFactorsByTypeTests() {
     val types = listOf(Log.TYPE, Pi.TYPE, Sqrt.TYPE, TestNumber.TYPE)