[lmi-commits] [5318] Remove markers for defects already resolved

[Greg Chicares]

Revision: 5318
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5318
Author:   chicares
Date:     2011-11-29 15:39:25 +0000 (Tue, 29 Nov 2011)
Log Message:
-----------
Remove markers for defects already resolved

Modified Paths:
--------------
    lmi/trunk/ihs_acctval.cpp

Modified: lmi/trunk/ihs_acctval.cpp
===================================================================
--- lmi/trunk/ihs_acctval.cpp   2011-11-29 15:24:22 UTC (rev 5317)
+++ lmi/trunk/ihs_acctval.cpp   2011-11-29 15:39:25 UTC (rev 5318)
@@ -451,13 +451,6 @@
             }
         }
 
-// The following two comment blocks will soon be expunged.
-//  - Irc7702_ and Irc7702A_ are appropriately initialized together
-//    here, soon after PerformSpecAmtStrategy() is called.
-
-    // TODO ?? We need to reconsider where the Irc7702 object gets created.
-    // Perhaps it should be when initial DB is first known? No, needed for
-    // solves. Then maybe we only need to move the call to Initialize7702?
     double annual_target_premium = GetModalTgtPrem
         (0
         ,mce_annual
@@ -557,10 +550,6 @@
         (yare_input_.Comments
         ,"idiosyncrasy_daily_interest_accounting"
         );
-
-/* TODO expunge?
-    PerformSpecAmtStrategy();
-*/
 }
 
 //============================================================================
@@ -981,39 +970,6 @@
         );
     AnnualTargetPrem = UnusedTargetPrem;
 
-// The following comment block will soon be expunged. At present,
-// BasicValues::GetTgtPrem() is used only by FindSpecAmt::operator().
-// Forgoing the specamt argument to GetTgtPrem() would require moving
-// it into class AccountValue, which would make it unsuitable for its
-// sole present purpose.
-
-/*
-// TODO ?? This code might be preferable if it worked correctly.
-//
-// Motivation for GetTgtPrem(): encapsulate calculations that need to
-// return the exact target premium, respecting all arcana such as
-// 'DB_TgtPremFixedAtIssue'.
-//
-// Defect in its implementation: specamt is passed as an argument, and
-// it's easy to get that wrong, as it is here. Real encapsulation
-// shouldn't need arguments.
-
-    MlyNoLapsePrem = GetTgtPrem
-        (Year
-        ,InvariantValues().SpecAmt[Year]
-        ,InvariantValues().DBOpt[Year].value()
-        ,mce_monthly
-        );
-
-    UnusedTargetPrem = GetTgtPrem
-        (Year
-        ,InvariantValues().SpecAmt[Year]
-        ,InvariantValues().DBOpt[Year].value()
-        ,mce_annual
-        );
-    AnnualTargetPrem = UnusedTargetPrem;
-*/
-
     if(0 == Year)
         {
         InvariantValues().InitTgtPrem = AnnualTargetPrem;