Optimized amount_t::in_place_truncate

2 files changed, 39 insertions, 3 deletions

diff --git a/src/amount.cc b/src/amount.cc
index 3a64577f..a16d287e 100644
--- a/src/amount.cc
+++ b/src/amount.cc
@@ -594,6 +594,44 @@ void amount_t::in_place_round()
   set_keep_precision(false);
 }
 
+void amount_t::in_place_truncate()
+{
+#if 1
+  if (! quantity)
+    throw_(amount_error, _("Cannot truncate an uninitialized amount"));
+
+  _dup();
+
+  DEBUG("amount.truncate",
+	"Truncating " << *this << " to precision " << display_precision());
+
+  std::ostringstream out;
+  stream_out_mpq(out, MP(quantity), display_precision());
+
+  scoped_array<char> buf(new char [out.str().length() + 1]);
+  std::strcpy(buf.get(), out.str().c_str());
+
+  char * q = buf.get();
+  for (char * p = q; *p != '\0'; p++, q++) {
+    if (*p == '.') p++;
+    if (p != q) *q = *p;
+  }
+  *q = '\0';
+
+  mpq_set_str(MP(quantity), buf.get(), 10);
+
+  mpz_ui_pow_ui(temp, 10, display_precision());
+  mpq_set_z(tempq, temp);
+  mpq_div(MP(quantity), MP(quantity), tempq);
+
+  DEBUG("amount.truncate", "Truncated = " << *this);
+#else
+  // This naive implementation is straightforward, but extremely inefficient
+  // as it requires parsing the commodity too, which might be fully annotated.
+  *this = amount_t(to_string());
+#endif
+}
+
 void amount_t::in_place_floor()
 {
   if (! quantity)
diff --git a/src/amount.h b/src/amount.h
index 5c1bca46..ae0e5a69 100644
--- a/src/amount.h
+++ b/src/amount.h
@@ -346,9 +346,7 @@ public:
     temp.in_place_truncate();
     return temp;
   }
-  void in_place_truncate() {
-    *this = amount_t(to_string());
-  }
+  void in_place_truncate();
     
   /** Yields an amount which has lost all of its extra precision, beyond what
       the display precision of the commodity would have printed. */