[lmi-commits] [6416] Eliminate an unused function argument

[Greg Chicares]

Revision: 6416
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=6416
Author:   chicares
Date:     2015-11-20 18:43:22 +0000 (Fri, 20 Nov 2015)
Log Message:
-----------
Eliminate an unused function argument

Modified Paths:
--------------
    lmi/trunk/ihs_avmly.cpp
    lmi/trunk/ihs_irc7702.cpp
    lmi/trunk/ihs_irc7702.hpp

Modified: lmi/trunk/ihs_avmly.cpp
===================================================================
--- lmi/trunk/ihs_avmly.cpp     2015-11-19 21:47:35 UTC (rev 6415)
+++ lmi/trunk/ihs_avmly.cpp     2015-11-20 18:43:22 UTC (rev 6416)
@@ -569,10 +569,10 @@
 //    if(!SolvingForGuarPremium && Solving || mce_run_gen_curr_sep_full == 
RunBasis_)
     if(Solving || mce_run_gen_curr_sep_full == RunBasis_)
         {
+        // Illustration-reg guaranteed premium ignores GPT limit.
         if(!SolvingForGuarPremium)
             {
-            double fake_cum_pmt = 0.0; // TODO ?? TAXATION !! Needs work.
-            Irc7702_->ProcessGptPmt(Year, GrossPmts[Month], fake_cum_pmt);
+            Irc7702_->ProcessGptPmt(Year, GrossPmts[Month]);
             }
         // Limit external 1035 first, then internal, as necessary to avoid
         // exceeding the guideline limit. This is what the customer would
@@ -1263,12 +1263,10 @@
         if(ee_pay_this_month)
             {
             double eepmt = PerformEePmtStrategy();
-            // Don't enforce the GPT premium limit when solving for
-            // illustration-reg guaranteed premium.
+            // Illustration-reg guaranteed premium ignores GPT limit.
             if(!SolvingForGuarPremium)
                 {
-                double fake_cum_pmt = 0.0; // TODO ?? TAXATION !! Needs work.
-                Irc7702_->ProcessGptPmt(Year, eepmt, fake_cum_pmt);
+                Irc7702_->ProcessGptPmt(Year, eepmt);
                 }
             EeGrossPmts[Month] += eepmt;
             GrossPmts  [Month] += eepmt;
@@ -1276,12 +1274,10 @@
         if(er_pay_this_month)
             {
             double erpmt = PerformErPmtStrategy();
-            // Don't enforce the GPT premium limit when solving for
-            // illustration-reg guaranteed premium.
+            // Illustration-reg guaranteed premium ignores GPT limit.
             if(!SolvingForGuarPremium)
                 {
-                double fake_cum_pmt = 0.0; // TODO ?? TAXATION !! Needs work.
-                Irc7702_->ProcessGptPmt(Year, erpmt, fake_cum_pmt);
+                Irc7702_->ProcessGptPmt(Year, erpmt);
                 }
             ErGrossPmts[Month] += erpmt;
             GrossPmts  [Month] += erpmt;
@@ -1293,12 +1289,10 @@
 
     if(0 == Year && 0 == Month && (Solving || mce_run_gen_curr_sep_full == 
RunBasis_))
         {
-        // Don't enforce the GPT premium limit when solving for
-        // illustration-reg guaranteed premium.
+        // Illustration-reg guaranteed premium ignores GPT limit.
         if(!SolvingForGuarPremium)
             {
-            double fake_cum_pmt = 0.0; // TODO ?? TAXATION !! Needs work.
-            Irc7702_->ProcessGptPmt(Year, Dumpin, fake_cum_pmt);
+            Irc7702_->ProcessGptPmt(Year, Dumpin);
             }
         EeGrossPmts[Month] += Dumpin;
         GrossPmts  [Month] += Dumpin;
@@ -2666,11 +2660,13 @@
 
     if(Solving || mce_run_gen_curr_sep_full == RunBasis_)
         {
+        // Illustration-reg guaranteed premium ignores GPT limit.
         if(!SolvingForGuarPremium)
             {
-            double fake_cum_pmt = 0.0; // TODO ?? TAXATION !! Needs work.
+            // TODO ?? TAXATION !! What if reference argument
+            // 'premiums_paid_increment' is modified?
             double premiums_paid_increment = -GrossWD;
-            Irc7702_->ProcessGptPmt(Year, premiums_paid_increment, 
fake_cum_pmt);
+            Irc7702_->ProcessGptPmt(Year, premiums_paid_increment);
             }
         }
 

Modified: lmi/trunk/ihs_irc7702.cpp
===================================================================
--- lmi/trunk/ihs_irc7702.cpp   2015-11-19 21:47:35 UTC (rev 6415)
+++ lmi/trunk/ihs_irc7702.cpp   2015-11-20 18:43:22 UTC (rev 6416)
@@ -216,11 +216,10 @@
 void Irc7702::ProcessGptPmt
     (int     a_Duration
     ,double& a_Pmt
-    ,double& // a_CumPmt
     )
 {
-// TODO ?? Duration and CumPmt args not yet used.
-// TAXATION !! Would Duration somehow be used for verification?
+// TAXATION !! Should Duration somehow be used for verification? and
+// if so, should it be maintained internally in this class?
     if(mce_gpt != Test7702)
         {
         return;

Modified: lmi/trunk/ihs_irc7702.hpp
===================================================================
--- lmi/trunk/ihs_irc7702.hpp   2015-11-19 21:47:35 UTC (rev 6415)
+++ lmi/trunk/ihs_irc7702.hpp   2015-11-20 18:43:22 UTC (rev 6416)
@@ -108,7 +108,6 @@
     void ProcessGptPmt
         (int                        a_Duration
         ,double&                    a_Pmt
-        ,double&                    a_CumPmt
         );
     // Returns forceout if any, else 0.0 . TAXATION !! But it's void.
     void ProcessAdjustableEvent