lmi-commits
[Top][All Lists]
Advanced

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

[lmi-commits] [lmi] master 1e5aad9 1/5: Remove invalidated comments abou


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 1e5aad9 1/5: Remove invalidated comments about invalidation
Date: Thu, 4 Aug 2016 13:10:37 +0000 (UTC)

branch: master
commit 1e5aad92bbed65be3e495ba3e9e454206c5b9874
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Remove invalidated comments about invalidation
    
    The removed comments documented problems with product-database cache
    invalidation that have recently been eradicated.
---
 census_view.cpp |    7 +------
 dbdict.cpp      |    7 +------
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/census_view.cpp b/census_view.cpp
index 4f0a2bb..32821c3 100644
--- a/census_view.cpp
+++ b/census_view.cpp
@@ -1160,12 +1160,7 @@ void CensusView::apply_changes
 
     // Probably this should be factored out into a member function
     // that's called elsewhere too--e.g., when a cell is read from
-    // file, or when a census is pasted. For this to work fully as
-    // desired, however, the DATABASE !! must be changed. Today,
-    // it caches exactly one product, and its cache-invalidation
-    // discipline isn't sufficiently strict. For now, applying the
-    // present technique elsewhere might well exacerbate crosstalk
-    // in a census that comprises more than one product.
+    // file, or when a census is pasted.
     typedef std::vector<Input>::iterator ii;
     for(ii j = class_parms().begin(); j != class_parms().end(); ++j)
         {
diff --git a/dbdict.cpp b/dbdict.cpp
index 045d5f5..3f25f49 100644
--- a/dbdict.cpp
+++ b/dbdict.cpp
@@ -449,12 +449,7 @@ void DBDictionary::Init(std::string const& filename)
         }
 }
 
-/// Save file, invalidating the cache.
-///
-/// If data are modified (by the GUI product editor) and saved under a
-/// new name, the cache must be invalidated. Otherwise, calling Init()
-/// with the name of the intact original file would fail to reload the
-/// unmodified data.
+/// Save a database file.
 
 void DBDictionary::WriteDB(std::string const& filename) const
 {



reply via email to

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