[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master aab6893 2/2: Use modified argument instead of
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master aab6893 2/2: Use modified argument instead of one of its addends |
Date: |
Mon, 13 Dec 2021 17:55:21 -0500 (EST) |
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;