[lmi-commits] [lmi] master aab6893 2/2: Use modified argument instead of one of its addends

[Greg Chicares]

branch: master
commit aab6893c831e91f5332f495d026b66dce805cdfc
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Use modified argument instead of one of its addends
    
    See:
      https://lists.nongnu.org/archive/html/lmi/2021-12/msg00001.html
    
    This comment:
      /// Non-tiered complements are added to each argument as needed.
    plainly seems to state that the intention was to treat all three
    'double' arguments as increments to non-tiered values and use the
    values thus incremented. This change has no effect on regression
    testing, which proves only that there's no test for it; however, in
    AccountValue::ApplyDynamicMandE(), the only place that function is
    called, a nonzero value of the argument raises a
      "Tiered investment management fee unimplemented."
    alarum, so any test for this feature would fail anyway.
---
 interest_rates.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/interest_rates.cpp b/interest_rates.cpp
index d063bdf..be6ac34 100644
--- a/interest_rates.cpp
+++ b/interest_rates.cpp
@@ -749,7 +749,7 @@ void InterestRates::DynamicMlySepAcctRate
                 ,dynamic_spread
                 ,SepAcctSpreadMethod_
                 ,SepAcctFloor_[year]
-                ,InvestmentManagementFee_[year]
+                ,AnnualSepAcctIMFRate
                 );
             }
             break;