[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 81f57a3 1/3: Resolve a question
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 81f57a3 1/3: Resolve a question |
Date: |
Sat, 21 Apr 2018 15:08:07 -0400 (EDT) |
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