[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master b85da3e 24/46: Clarify the comment about Cens

From: Greg Chicares
Subject: [lmi-commits] [lmi] master b85da3e 24/46: Clarify the comment about CensusViewDataViewModel wrappers
Date: Wed, 22 Jul 2020 11:05:13 -0400 (EDT)

branch: master
commit b85da3ec41daa4e078388f63c186d6ec4b67eece
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Vadim Zeitlin <vadim@tt-solutions.com>

    Clarify the comment about CensusViewDataViewModel wrappers
    Explain why do we do it like this, rather than declare
    CensusViewDataViewModel earlier.
 census_view.cpp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/census_view.cpp b/census_view.cpp
index 6f0d6a7..484241e 100644
--- a/census_view.cpp
+++ b/census_view.cpp
@@ -938,8 +938,13 @@ inline std::vector<std::string> const& 
CensusViewDataViewModel::all_headers() co
-// Declare the functions here, but implement after the CensusViewGridTable
-// declaration to avoid the "use of undefined type" error.
+// Declare free functions corresponding to their namesakes in 
+// that can be used in the implementation of the custom editors below, which
+// are defined before CensusViewGridTable is declared.
+// This seems preferable to intermixing the declarations and definitions, as it
+// allows to declare (and define) all the classes in consistent bottom-to-top
+// order -- but at a price of having these not really indispensable wrappers.
 // Get the cell value from the table.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]