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

[Greg Chicares]

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
 namespace
 {
 
-// 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 
CensusViewGridTable
+// 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.