From cce65b8dd2f9cb22a20acb891a8a83cdfce90c7b Mon Sep 17 00:00:00 2001
From: John Wiegley <johnw@newartisans.com>
Date: Sat, 31 Jan 2009 04:58:16 -0400
Subject: Added a more efficient amount_t::operator==, which changed semantics
 slightly.

---
 src/amount.cc | 12 ++++++++++++
 src/amount.h  |  6 ------
 2 files changed, 12 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/amount.cc b/src/amount.cc
index 653272ed..bc8b0b6f 100644
--- a/src/amount.cc
+++ b/src/amount.cc
@@ -259,6 +259,18 @@ int amount_t::compare(const amount_t& amt) const
   return mpq_cmp(MP(quantity), MP(amt.quantity));
 }
 
+bool amount_t::operator==(const amount_t& amt) const
+{
+  if ((quantity && ! amt.quantity) || (! quantity && amt.quantity))
+    return false;
+  else if (! quantity && ! amt.quantity)
+    return true;
+  else if (commodity() != amt.commodity())
+    return false;
+
+  return mpq_equal(MP(quantity), MP(amt.quantity));
+}
+
 
 amount_t& amount_t::operator+=(const amount_t& amt)
 {
diff --git a/src/amount.h b/src/amount.h
index ef9fc4b0..d5c759db 100644
--- a/src/amount.h
+++ b/src/amount.h
@@ -781,12 +781,6 @@ inline std::istream& operator>>(std::istream& in, amount_t& amt) {
 
 namespace ledger {
 
-inline bool amount_t::operator==(const amount_t& amt) const {
-  if (commodity() != amt.commodity())
-    return false;
-  return compare(amt) == 0;
-}
-
 inline commodity_t& amount_t::commodity() const {
   return has_commodity() ? *commodity_ : *current_pool->null_commodity;
 }
-- 
cgit v1.2.3