If AMOUNT_PARSE_NO_MIGRATE is set, do not set a commodity's precision even if

this is the first time a commodity is being seen.  The reason is that it's
quite possible (especially in bug reports) for the cost of a priced
transaction to contain the first occurrence of a commodity.

1 files changed, 3 insertions, 4 deletions

diff --git a/amount.cc b/amount.cc
index 69d26477..1ac2f407 100644
--- a/amount.cc
+++ b/amount.cc
@@ -1170,14 +1170,13 @@ void amount_t::parse(std::istream& in, unsigned char flags)
 
   // Set the commodity's flags and precision accordingly
 
-  if (newly_created || ! (flags & AMOUNT_PARSE_NO_MIGRATE)) {
+  if (! (flags & AMOUNT_PARSE_NO_MIGRATE)) {
     commodity().add_flags(comm_flags);
     if (quantity->prec > commodity().precision())
       commodity().set_precision(quantity->prec);
-  }
-
-  if (flags & AMOUNT_PARSE_NO_MIGRATE)
+  } else {
     quantity->flags |= BIGINT_KEEP_PREC;
+  }
 
   // Now we have the final number.  Remove commas and periods, if
   // necessary.