[lmi-commits] [lmi] master 5d29307 2/2: Document a defect

[Greg Chicares]

branch: master
commit 5d29307d6a7eef746fe267f981274f8ff4d74820
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Document a defect
---
 ill_reg_narr_summary.mst | 12 ++++++++++++
 pdf_command_wx.cpp       |  3 ++-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/ill_reg_narr_summary.mst b/ill_reg_narr_summary.mst
index dff0634..23b4f5d 100644
--- a/ill_reg_narr_summary.mst
+++ b/ill_reg_narr_summary.mst
@@ -27,6 +27,18 @@
 
 <font size="-1">
 
+{{!
+PDF !! The only effect of the 'GroupCarveout' conditional below is
+to produce the string:
+  "group flexible premium adjustable life insurance contract"
+if 'PolicyLegalName' exactly matches this string:
+  "Group Flexible Premium Adjustable Life Insurance Certificate"
+although probably "certificate" is desired instead of "contract".
+If it is actually considered useful to have two distinct strings
+that differ only in their capitalization, then both should reside
+in the '.policy' file.
+}}
+
 <p>
 {{^SinglePremium}}
     {{PolicyMktgName}} is a
diff --git a/pdf_command_wx.cpp b/pdf_command_wx.cpp
index 45b21c8..7307c14 100644
--- a/pdf_command_wx.cpp
+++ b/pdf_command_wx.cpp
@@ -975,7 +975,8 @@ class pdf_illustration : protected html_interpolator, 
protected pdf_writer_wx
             );
 
         // PDF !! Conditions of this ilk should become distinct entities in
-        // the product files.
+        // the product files--or in this case, eliminated altogether: this
+        // condition is used only in one place, in a really silly way.
         auto const& policy_name = invar.PolicyLegalName;
         add_variable
             ("GroupCarveout"