lmi-commits
[Top][All Lists]
Advanced

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

[lmi-commits] [lmi] master 7b804b9 1/2: Change the way 'CsoEra' is summa


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 7b804b9 1/2: Change the way 'CsoEra' is summarized for composites
Date: Wed, 21 Aug 2019 11:01:28 -0400 (EDT)

branch: master
commit 7b804b934f0828e3403444e7d150911dbe504e61
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Change the way 'CsoEra' is summarized for composites
---
 ledger_invariant.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/ledger_invariant.cpp b/ledger_invariant.cpp
index 7020438..9b3b332 100644
--- a/ledger_invariant.cpp
+++ b/ledger_invariant.cpp
@@ -579,10 +579,11 @@ LedgerInvariant& LedgerInvariant::PlusEq(LedgerInvariant 
const& a_Addend)
     PolicyForm                    = a_Addend.PolicyForm;
     PolicyMktgName                = a_Addend.PolicyMktgName;
     PolicyLegalName               = a_Addend.PolicyLegalName;
-    // This doesn't seem appropriate for a composite, which may
-    // include contracts issued in different CSO eras:
-//  CsoEra                        = a_Addend.CsoEra;
-    CsoEra                        = "[composite]";
+    // It is inappropriate to "summarize" CsoEra for a composite that
+    // includes contracts issued in different CSO eras. Choosing the
+    // last cell's value does the "right" thing when all contracts are
+    // of the same CSO era, as is most often the case.
+    CsoEra                        = a_Addend.CsoEra;
     InsCoShortName                = a_Addend.InsCoShortName;
     InsCoName                     = a_Addend.InsCoName;
     InsCoAddr                     = a_Addend.InsCoAddr;



reply via email to

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