better coverage for irrational fractions equality comparison

ohhmm · Feb 14, 2025 · 198143f · 198143f
1 parent ab6287b
commit 198143f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/omnn/math/Fraction.cpp b/omnn/math/Fraction.cpp
@@ -78,9 +78,9 @@ using namespace omnn::math;
                     auto bothAreRational = (IsRational() && fraction.IsRational()) == YesNoMaybe::Yes;
                     if (bothAreRational) {
                         equal = operator a_rational() == static_cast<a_rational>(fraction);
-                    } else { // TODO: FIXME: both irrationals hangs on macos
-                        //OptimizeOn on;
-                        //equal = numerator() * fraction.denominator() == fraction.numerator() * denominator();
+                    } else { // both are irrational
+                        OptimizeOn on;
+                        equal = numerator() * fraction.denominator() == fraction.numerator() * denominator();
                     }
                 }
             }