BIND 10 trac2209, updated. 414a83fefbb7374f31f362e80efef262722660eb [2209] Rename ZONE_RELOADED

[BIND 10 source code commits]

The branch, trac2209 has been updated
       via  414a83fefbb7374f31f362e80efef262722660eb (commit)
      from  6ead8f2bc21f09b34f842dda074d121a9f511ed5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 414a83fefbb7374f31f362e80efef262722660eb
Author: Michal 'vorner' Vaner <michal.vaner at nic.cz>
Date:   Fri Oct 26 15:55:33 2012 +0200

    [2209] Rename ZONE_RELOADED
    
    Since it is not used for reloading a zone only. It is now ZONE_SUCCESS.

-----------------------------------------------------------------------

Summary of changes:
 src/bin/auth/command.cc                       |    2 +-
 src/lib/datasrc/client_list.cc                |    2 +-
 src/lib/datasrc/client_list.h                 |    5 +++--
 src/lib/datasrc/tests/client_list_unittest.cc |    8 ++++----
 4 files changed, 9 insertions(+), 8 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/auth/command.cc b/src/bin/auth/command.cc
index efd60f2..84c1f7d 100644
--- a/src/bin/auth/command.cc
+++ b/src/bin/auth/command.cc
@@ -203,7 +203,7 @@ public:
         }
 
         switch (list->reload(origin)) {
-            case ConfigurableClientList::ZONE_RELOADED:
+            case ConfigurableClientList::ZONE_SUCCESS:
                 // Everything worked fine.
                 LOG_DEBUG(auth_logger, DBG_AUTH_OPS, AUTH_LOAD_ZONE)
                     .arg(zone_class).arg(origin);
diff --git a/src/lib/datasrc/client_list.cc b/src/lib/datasrc/client_list.cc
index 6813f3c..21b147e 100644
--- a/src/lib/datasrc/client_list.cc
+++ b/src/lib/datasrc/client_list.cc
@@ -435,7 +435,7 @@ ConfigurableClientList::getCachedZoneWriter(const Name& name) {
         load_action = boost::bind(loadZoneDataFromFile, _1, rrclass_, name,
                                   filename);
     }
-    return (ZoneWriterPair(ZONE_RELOADED,
+    return (ZoneWriterPair(ZONE_SUCCESS,
                            ZoneWriterPtr(
                                result.info->cache_->getZoneTableSegment().
                                getZoneWriter(load_action, name, rrclass_))));
diff --git a/src/lib/datasrc/client_list.h b/src/lib/datasrc/client_list.h
index ecb20a5..a6a42b2 100644
--- a/src/lib/datasrc/client_list.h
+++ b/src/lib/datasrc/client_list.h
@@ -273,7 +273,8 @@ public:
         CACHE_DISABLED,     ///< The cache is not enabled in this list.
         ZONE_NOT_CACHED,    ///< Zone is served directly, not from cache.
         ZONE_NOT_FOUND,     ///< Zone does not exist or not cached.
-        ZONE_RELOADED       ///< The zone was successfully reloaded.
+        ZONE_SUCCESS        ///< The zone was successfully reloaded or
+                            ///  the writer provided.
     };
 
     /// \brief Reloads a cached zone.
@@ -309,7 +310,7 @@ public:
     /// \param zone The origin of the zone to reload.
     /// \return The result has two parts. The first one is a status describing
     ///     if it worked or not (and in case it didn't, also why). If the
-    ///     status is ZONE_RELOADED, the second part contains a shared pointer
+    ///     status is ZONE_SUCCESS, the second part contains a shared pointer
     ///     to the writer. If the status is anything else, the second part is
     ///     NULL.
     /// \throw DataSourceError or anything else that the data source
diff --git a/src/lib/datasrc/tests/client_list_unittest.cc b/src/lib/datasrc/tests/client_list_unittest.cc
index 4647f2b..0c95350 100644
--- a/src/lib/datasrc/tests/client_list_unittest.cc
+++ b/src/lib/datasrc/tests/client_list_unittest.cc
@@ -870,7 +870,7 @@ ConfigurableClientList::ReloadResult
 ReloadTest<WriterUpdateType>::doReload(const Name& origin) {
     ConfigurableClientList::ZoneWriterPair
         result(list_->getCachedZoneWriter(origin));
-    if (result.first == ConfigurableClientList::ZONE_RELOADED) {
+    if (result.first == ConfigurableClientList::ZONE_SUCCESS) {
         // Can't use ASSERT_NE here, it would wan't to return(), which
         // it can't in non-void function.
         if (result.second) {
@@ -878,7 +878,7 @@ ReloadTest<WriterUpdateType>::doReload(const Name& origin) {
             result.second->install();
             result.second->cleanup();
         } else {
-            ADD_FAILURE() << "getCachedZoneWriter returned ZONE_RELOADED, "
+            ADD_FAILURE() << "getCachedZoneWriter returned ZONE_SUCCESS, "
                 "but the writer is NULL";
         }
     } else {
@@ -902,7 +902,7 @@ TYPED_TEST(ReloadTest, reloadSuccess) {
     EXPECT_EQ(ZoneFinder::NXRRSET,
               this->list_->find(name).finder_->find(name, RRType::NS())->code);
     // Now reload the full zone. It should be there now.
-    EXPECT_EQ(ConfigurableClientList::ZONE_RELOADED, this->doReload(name));
+    EXPECT_EQ(ConfigurableClientList::ZONE_SUCCESS, this->doReload(name));
     EXPECT_EQ(ZoneFinder::SUCCESS,
               this->list_->find(name).finder_->find(name, RRType::NS())->code);
 }
@@ -1035,7 +1035,7 @@ TYPED_TEST(ReloadTest, reloadMasterFile) {
     f << "nosuchdomain.\t\t3600\tIN\tTXT\ttest" << std::endl;
     f.close();
     // Do the reload.
-    EXPECT_EQ(ConfigurableClientList::ZONE_RELOADED, this->doReload(Name(".")));
+    EXPECT_EQ(ConfigurableClientList::ZONE_SUCCESS, this->doReload(Name(".")));
     // It is here now.
     EXPECT_EQ(ZoneFinder::SUCCESS,
               this->list_->find(Name(".")).finder_->find(Name("nosuchdomain"),