[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;