[lmi-commits] [lmi] valyuta/004 1f7a1c7 3/6: Note some ideas for future consideration

[Greg Chicares]

branch: valyuta/004
commit 1f7a1c7dcdc90f2c90cd12a375ff7d24e8f11f4b
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Note some ideas for future consideration
---
 currency.hpp  | 3 +++
 ihs_avmly.cpp | 1 +
 2 files changed, 4 insertions(+)

diff --git a/currency.hpp b/currency.hpp
index a835d1e..753c732 100644
--- a/currency.hpp
+++ b/currency.hpp
@@ -86,6 +86,9 @@ class currency
     // The extra, ignored argument ensures that this special-purpose
     // ctor ambiguates no other. (But that doesn't matter now that
     // there is no other non-default ctor.)
+    //
+    // CURRENCY !! Probably the first argument should be asserted to
+    // have an exact integral value.
     explicit currency(data_type c, raw_cents) : m_ {c} {}
 // Converting ctors are preferably avoided:
 #if 0 // Apparently unused.
diff --git a/ihs_avmly.cpp b/ihs_avmly.cpp
index b754b97..d9a51a9 100644
--- a/ihs_avmly.cpp
+++ b/ihs_avmly.cpp
@@ -2085,6 +2085,7 @@ void AccountValue::TxTakeSepAcctLoad()
         }
 
     SepAcctLoad = round_interest_credit().c(YearsSepAcctLoadRate * AVSepAcct);
+    // Does this seem right? Mightn't it take a sepacct load from the genacct?
     process_deduction(SepAcctLoad);
     YearsTotalSepAcctLoad += SepAcctLoad;
     Dcv -= SepAcctLoad.d();