From df7158c0ee9cd6fde0183b2841d1c918f5c8ab7b Mon Sep 17 00:00:00 2001
From: Matthew Hoops
Date: Mon, 6 Sep 2010 14:59:13 +0000
Subject: COMMON: Add a debugPrint function to Rational

svn-id: r52593
---
 common/rational.cpp | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'common/rational.cpp')

diff --git a/common/rational.cpp b/common/rational.cpp
index 163044f349..38aae94def 100644
--- a/common/rational.cpp
+++ b/common/rational.cpp
@@ -22,6 +22,7 @@
  * $Id$
  */
 
+#include "common/debug.h"
 #include "common/rational.h"
 #include "common/util.h"
 #include "common/algorithm.h"
@@ -289,6 +290,10 @@ const Rational operator/(int left, const Rational &right) {
 	return tmp;
 }
 
+void Rational::debugPrint(int debuglevel, const char *caption) const {
+	debug(debuglevel, "%s %d/%d", caption, _num, _denom);
+}
+
 bool operator==(int left, const Rational &right) {
 	return right == left;
 }
-- 
cgit v1.2.3


From 8f1143bfdc972d86cf5fbbf85a0084a9d0c71447 Mon Sep 17 00:00:00 2001
From: Willem Jan Palenstijn
Date: Tue, 7 Sep 2010 09:02:46 +0000
Subject: COMMON: Remove Rational::operator int/double

This prevents accidental implicit rounding
and might also fix compilation on AmigaOS4 (bug #3060981).

svn-id: r52616
---
 common/rational.cpp | 8 --------
 1 file changed, 8 deletions(-)

(limited to 'common/rational.cpp')

diff --git a/common/rational.cpp b/common/rational.cpp
index 38aae94def..999dc7feb8 100644
--- a/common/rational.cpp
+++ b/common/rational.cpp
@@ -258,14 +258,6 @@ frac_t Rational::toFrac() const {
 	return (_num * FRAC_ONE) / _denom;
 }
 
-Rational::operator int() const {
-	return toInt();
-}
-
-Rational::operator double() const {
-	return toDouble();
-}
-
 const Rational operator+(int left, const Rational &right) {
 	Rational tmp = right;
 	tmp += left;
-- 
cgit v1.2.3


From a71f3aa99c3464665f45571822cf1eada2693cfe Mon Sep 17 00:00:00 2001
From: Sven Hesse
Date: Sat, 11 Sep 2010 10:03:31 +0000
Subject: COMMON: Fix int-Rational operators

svn-id: r52675
---
 common/rational.cpp | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

(limited to 'common/rational.cpp')

diff --git a/common/rational.cpp b/common/rational.cpp
index 999dc7feb8..f55c2dcfe3 100644
--- a/common/rational.cpp
+++ b/common/rational.cpp
@@ -259,26 +259,26 @@ frac_t Rational::toFrac() const {
 }
 
 const Rational operator+(int left, const Rational &right) {
-	Rational tmp = right;
-	tmp += left;
+	Rational tmp(left);
+	tmp += right;
 	return tmp;
 }
 
 const Rational operator-(int left, const Rational &right) {
-	Rational tmp = right;
-	tmp -= left;
+	Rational tmp(left);
+	tmp -= right;
 	return tmp;
 }
 
 const Rational operator*(int left, const Rational &right) {
-	Rational tmp = right;
-	tmp *= left;
+	Rational tmp(left);
+	tmp *= right;
 	return tmp;
 }
 
 const Rational operator/(int left, const Rational &right) {
-	Rational tmp = right;
-	tmp /= left;
+	Rational tmp(left);
+	tmp /= right;
 	return tmp;
 }
 
-- 
cgit v1.2.3