[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master ff118b3 5/8: Assert that a refactoring will b
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master ff118b3 5/8: Assert that a refactoring will be safe |
Date: |
Wed, 12 Sep 2018 12:08:06 -0400 (EDT) |
branch: master
commit ff118b392a1ea9600aa9ec9aa14b89b8534cc50e
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>
Assert that a refactoring will be safe
---
ledger_pdf_generator_wx.cpp | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/ledger_pdf_generator_wx.cpp b/ledger_pdf_generator_wx.cpp
index 4beb5d5..6d262e2 100644
--- a/ledger_pdf_generator_wx.cpp
+++ b/ledger_pdf_generator_wx.cpp
@@ -1497,6 +1497,15 @@ class numeric_summary_table_cell
}
}
+ if(is_last_row)
+ {
+ // If this assertion always succeeds, then the code
+ // that establishes this invariant above, and the
+ // assertion, can be removed.
+ LMI_ASSERT(summary_age_string ==
visible_values.at(column_policy_year));
+ visible_values.at(column_policy_year) =
summary_age_string;
+ }
+
table_gen.output_row(pos_y, visible_values);
}
break;
- [lmi-commits] [lmi] master updated (d8aa128 -> 7bf948b), Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master 039924c 6/8: Refactor to prepare for a larger refactoring, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master ff118b3 5/8: Assert that a refactoring will be safe,
Greg Chicares <=
- [lmi-commits] [lmi] master 7bf948b 8/8: Refactor for clarity, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master 0d7d4d5 4/8: Simplify some special-case logic to make it easier to relocate, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master 99930af 1/8: Declare a variable only in the scope where it's used, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master 039c1fd 2/8: Declare a variable only in the scope where it's used, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master a049a0d 3/8: Forestall latent off-by-one errors, Greg Chicares, 2018/09/12
- [lmi-commits] [lmi] master 1445c8d 7/8: Merge two identical blocks into a single function, Greg Chicares, 2018/09/12