From 63554274d8ca1b51b47da55c978c3354a91567bc Mon Sep 17 00:00:00 2001 From: David Gow Date: Thu, 7 Aug 2014 22:30:37 +0800 Subject: [PATCH] "fixed" Rational::ToDouble(). Previously if the numerator or denominator was > the range of a double, Rational.ToDouble() would return NaN. We now check for this case, and discard precision accordingly. --- src/rational.h | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/rational.h b/src/rational.h index bbd4c30..b019505 100644 --- a/src/rational.h +++ b/src/rational.h @@ -10,6 +10,8 @@ #include #include "arbint.h" #include "gmpint.h" +#include +#include namespace IPDF { @@ -171,7 +173,13 @@ struct Rational double ToDouble() const { - return (double)P/(double)Q; + T num = P, denom = Q; + while (Tabs(num) > T(DBL_MAX)) + { + num /= T(16); + denom /= T(16); + } + return ((double)(num))/((double)(denom)); } bool CheckAccuracy(double d, const char * msg, double threshold = 1e-3) const { -- 2.20.1