From: Michal 'vorner' Vaner <michal.vaner@nic.cz>
Date: Thu, 10 Jan 2013 15:23:13 +0000 (+0100)
Subject: [2436] Rephrase a TODO comment
X-Git-Tag: bind10-1.0.0-rc-release~99^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f8fb3f6f2f6f42888be0cd4e8c9df9502c944081;p=thirdparty%2Fkea.git

[2436] Rephrase a TODO comment

So it is a little bit clearer.
---

diff --git a/src/lib/datasrc/tests/zone_loader_unittest.cc b/src/lib/datasrc/tests/zone_loader_unittest.cc
index 3c5e502c55..1f6f07e40f 100644
--- a/src/lib/datasrc/tests/zone_loader_unittest.cc
+++ b/src/lib/datasrc/tests/zone_loader_unittest.cc
@@ -186,10 +186,12 @@ protected:
     {}
     void prepareSource(const Name& zone, const char* filename) {
         // TODO:
-        // Currently, load uses an implementation unrelated to the ZoneLoader.
-        // In the long term, the method will probably be deprecated. At that
-        // time, we should probably prepare the data in some other way (using
-        // sqlite3 or something). This is simpler for now.
+        // Currently, source_client_ is of InMemoryClient and its load()
+        // uses a different code than the ZoneLoader (so we can cross-check
+        // the implementations). Currently, the load() doesn't perform any
+        // post-load checks. It will change in #2499, at which point the
+        // loading may start failing depending on details of the test data. We
+        // should prepare the data by some different method then.
         source_client_.load(zone, string(TEST_DATA_DIR) + "/" + filename);
     }
 private: