[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master ff9d3bc 3/7: Rename a member function for cla
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master ff9d3bc 3/7: Rename a member function for clarity |
Date: |
Sun, 13 May 2018 19:45:54 -0400 (EDT) |
branch: master
commit ff9d3bc176eaada147231d2844195d0c9b004183
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>
Rename a member function for clarity
---
wx_table_generator.cpp | 12 ++++++------
wx_table_generator.hpp | 2 +-
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/wx_table_generator.cpp b/wx_table_generator.cpp
index 5cb5bc9..4fac20b 100644
--- a/wx_table_generator.cpp
+++ b/wx_table_generator.cpp
@@ -36,7 +36,7 @@
// accessible? But their values can also be changed by clients, and
// isn't that undesirable?
//
-// In wx_table_generator::do_output_values():
+// In wx_table_generator::do_output_single_row():
// if(align_right_)
// if(ci.is_centered_)
// it seems that right-alignment is a quasi-global, while
@@ -251,7 +251,7 @@ wx_table_generator::wx_table_generator
compute_column_widths();
// Set a pen with 0 width to get the thin lines, and round cap style for
the
- // different segments drawn in do_output_values() to seamlessly combine
+ // different segments drawn in do_output_single_row() to seamlessly combine
// into a single line.
wxPen pen(*wxBLACK, 0);
pen.SetCap(wxCAP_ROUND);
@@ -637,7 +637,7 @@ void wx_table_generator::compute_column_widths()
}
}
-void wx_table_generator::do_output_values
+void wx_table_generator::do_output_single_row
(int& pos_x
,int& pos_y
,std::vector<std::string> const& values
@@ -795,7 +795,7 @@ void wx_table_generator::output_header
// The distance from the font's descender line to its ascender
// line must not exceed the distance between lines.
LMI_ASSERT(dc_.GetCharHeight() <= row_height());
- // do_output_values(), called below, uses a cached char_height_
+ // do_output_single_row(), called below, uses a cached char_height_
// that is assumed not to differ from the bold GetCharHeight().
LMI_ASSERT(dc_.GetCharHeight() == char_height_);
}
@@ -831,7 +831,7 @@ void wx_table_generator::output_header
,headers_by_line.begin() + (1 + line) * num_columns
);
x = left_margin_;
- do_output_values(x, *pos_y, nth_line);
+ do_output_single_row(x, *pos_y, nth_line);
}
// Finally draw the separators above and (a double one) below them.
@@ -897,7 +897,7 @@ void wx_table_generator::output_row
)
{
int x = left_margin_;
- do_output_values(x, *pos_y, values);
+ do_output_single_row(x, *pos_y, values);
if(draw_separators_)
{
diff --git a/wx_table_generator.hpp b/wx_table_generator.hpp
index 64d5293..a670065 100644
--- a/wx_table_generator.hpp
+++ b/wx_table_generator.hpp
@@ -116,7 +116,7 @@ class wx_table_generator
void do_output_horz_separator(int x1, int x2, int y );
void do_output_vert_separator(int x , int y1, int y2);
- void do_output_values
+ void do_output_single_row
(int& pos_x
,int& pos_y
,std::vector<std::string> const& values
- [lmi-commits] [lmi] master updated (45b086a -> a738b9f), Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master 5fd8c4b 6/7: Name identical variables identically, Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master a738b9f 7/7: Use identical variables identically, Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master de4de8a 2/7: Use same argument names in implementation as in header, Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master 9aa7748 1/7: Correct a misleading comment, Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master ff9d3bc 3/7: Rename a member function for clarity,
Greg Chicares <=
- [lmi-commits] [lmi] master 06d9a8c 5/7: Pass modifiable arguments by reference, Greg Chicares, 2018/05/14
- [lmi-commits] [lmi] master a3a294f 4/7: Pass modifiable arguments by reference, Greg Chicares, 2018/05/14