lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master 2a2d275 6/8: Expunge vestiges of an unwanted


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 2a2d275 6/8: Expunge vestiges of an unwanted ctor [411]
Date: Mon, 30 Jan 2017 02:49:05 +0000 (UTC)

branch: master
commit 2a2d27572a2c3b6eededc650b8db590133cb9f3c
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Expunge vestiges of an unwanted ctor [411]
---
 input_sequence.cpp |   20 --------------------
 input_sequence.hpp |   11 +++++------
 2 files changed, 5 insertions(+), 26 deletions(-)

diff --git a/input_sequence.cpp b/input_sequence.cpp
index 7bbdbb4..82a08c9 100644
--- a/input_sequence.cpp
+++ b/input_sequence.cpp
@@ -271,26 +271,6 @@ InputSequence::InputSequence
 
 InputSequence::~InputSequence() = default;
 
-/*
-// TODO ?? Want a similar function for std::vector<string>, for enumerators?
-InputSequence::InputSequence
-    (double value_to_retirement
-    ,double value_from_retirement
-    )
-{
-// TODO ?? This loses the variable nature of retirement age.
-    ValueInterval to_retirement;
-    to_retirement.value_number     = value_to_retirement;
-    intervals.push_back(to_retirement);
-
-    ValueInterval from_retirement;
-    from_retirement.value_number     = value_from_retirement;
-    intervals.push_back(from_retirement);
-
-    realize_vector();
-}
-*/
-
 void InputSequence::realize_vector()
 {
     // Post-construction invariants.
diff --git a/input_sequence.hpp b/input_sequence.hpp
index 8521a0c..929d9db 100644
--- a/input_sequence.hpp
+++ b/input_sequence.hpp
@@ -31,15 +31,15 @@
 // meaning can be resolved only in the context of each life. It cannot
 // be expressed as a simple vector of values:
 //   premium: 10000, 10000, ...[how many?], 0, 0, ...[how many?]
-// At present, lmi offers separate pre- and post-retirement fields for
-// such inputs, and a simple vector of values to accommodate varying
+// A long time ago, lmi offered separate pre- and post-retirement fields
+// for such inputs, and a simple vector of values to accommodate varying
 // values that don't fit the [issue, retirement), [retirement, maturity)
 // paradigm, such as premiums following the exact pattern
 //   premium: 50000 25000 25000 10000 10000...
-// but these pardigms are disjoint, so it cannot easily accomodate
+// but these pardigms are disjoint, so it could not easily accomodate
 //   premium: 50000; 25000 [1, 3); 10000 [3, retirement); 0
-// Applying such a rule across a census of 1000 lives requires manually
-// editing each life, which is unacceptably tedious.
+// Applying such a rule across a census of 1000 lives required manually
+// editing each life, which was unacceptably tedious.
 
 // Extract the grammar from lines matching the regexp _// GRAMMAR_ in
 // the implementation file.
@@ -255,7 +255,6 @@ class LMI_SO InputSequence
         (std::vector<double> const&
         ,std::vector<std::string> const&
         );
-    InputSequence(double value_to_retirement, double value_from_retirement);
     ~InputSequence();
 
     // SOMEDAY !! This probably ought to return a reference.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]