[lmi-commits] [lmi] master 81f57a3 1/3: Resolve a question

[Greg Chicares]

branch: master
commit 81f57a3a5d98d7a80f628c96077268f879853fa6
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Resolve a question
    
    Renamed 'set_header_font', which sounded like a function, but was really
    a variable. See:
      https://lists.nongnu.org/archive/html/lmi/2018-04/msg00088.html
---
 wx_table_generator.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/wx_table_generator.cpp b/wx_table_generator.cpp
index ba17785..d776cde 100644
--- a/wx_table_generator.cpp
+++ b/wx_table_generator.cpp
@@ -90,12 +90,10 @@ void wx_table_generator::add_column
         }
     else
         {
-        wxDCFontChanger set_header_font(dc_);
+        wxDCFontChanger header_font_setter(dc_);
         if(use_bold_headers_)
             {
-// set(get()) sounds like a do-nothing operation;
-// how can it make a font bold?
-            set_header_font.Set(get_header_font());
+            header_font_setter.Set(get_header_font());
             }
 
         // Set width to the special value of 0 for the variable width columns.
@@ -546,10 +544,10 @@ void wx_table_generator::output_header
 
     do_compute_column_widths_if_necessary();
 
-    wxDCFontChanger set_header_font(dc_);
+    wxDCFontChanger header_font_setter(dc_);
     if(use_bold_headers_)
         {
-        set_header_font.Set(get_header_font());
+        header_font_setter.Set(get_header_font());
         }
 
     // Split headers in single lines and fill up the entire columns*lines 2D