BIND 10 trac471, updated. 61e7f493f8f44c7ccb8f2d9ef0186dd15e499097 [trac471] address review comments

[BIND 10 source code commits]

The branch, trac471 has been updated
       via  61e7f493f8f44c7ccb8f2d9ef0186dd15e499097 (commit)
      from  13a3fcfa3f8f256f5185faffededd36bcd03e5b5 (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 61e7f493f8f44c7ccb8f2d9ef0186dd15e499097
Author: Jelte Jansen <jelte at isc.org>
Date:   Wed Mar 16 14:19:04 2011 +0100

    [trac471] address review comments

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

Summary of changes:
 src/bin/resolver/resolver.cc                  |    5 -----
 src/bin/resolver/resolver.h                   |    8 ++++----
 src/lib/config/module_spec.cc                 |   14 ++++++--------
 src/lib/config/tests/module_spec_unittests.cc |    4 ++++
 4 files changed, 14 insertions(+), 17 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/resolver/resolver.cc b/src/bin/resolver/resolver.cc
index ad7ed3a..7c70395 100644
--- a/src/bin/resolver/resolver.cc
+++ b/src/bin/resolver/resolver.cc
@@ -585,11 +585,6 @@ Resolver::updateConfig(ConstElementPtr config) {
 }
 
 void
-Resolver::setConfigured() {
-    configured_ = true;
-}
-
-void
 Resolver::setForwardAddresses(const AddressList& addresses)
 {
     impl_->setForwardAddresses(addresses, dnss_);
diff --git a/src/bin/resolver/resolver.h b/src/bin/resolver/resolver.h
index ba2cc37..002e58b 100644
--- a/src/bin/resolver/resolver.h
+++ b/src/bin/resolver/resolver.h
@@ -119,11 +119,11 @@ public:
     asiolink::SimpleCallback* getCheckinProvider() { return (checkin_); }
 
     /**
-     * \brief Tell the Resolver that is should has already been
-     *        configured (used by updateConfig() and tests, so
-     *        that it will only set some defaults the first time
+     * \brief Tell the Resolver that is has already been configured
+     *        so that it will only set some defaults the first time
+     *        (used by updateConfig() and tests)
      */
-    void setConfigured();
+    void setConfigured() { configured_ = true; };
 
     /**
      * \brief Specify the list of upstream servers.
diff --git a/src/lib/config/module_spec.cc b/src/lib/config/module_spec.cc
index 5363b04..1621fe3 100644
--- a/src/lib/config/module_spec.cc
+++ b/src/lib/config/module_spec.cc
@@ -373,19 +373,17 @@ ModuleSpec::validateSpecList(ConstElementPtr spec, ConstElementPtr data,
     BOOST_FOREACH(maptype m, data->mapValue()) {
         bool found = false;
         // Ignore 'version' as a config element
-        if (m.first.compare("version") == 0) {
-            found = true;
-        } else {
+        if (m.first.compare("version") != 0) {
             BOOST_FOREACH(ConstElementPtr cur_spec_el, spec->listValue()) {
                 if (cur_spec_el->get("item_name")->stringValue().compare(m.first) == 0) {
                     found = true;
                 }
             }
-        }
-        if (!found) {
-            validated = false;
-            if (errors) {
-                errors->add(Element::create("Unknown item " + m.first));
+            if (!found) {
+                validated = false;
+                if (errors) {
+                    errors->add(Element::create("Unknown item " + m.first));
+                }
             }
         }
     }
diff --git a/src/lib/config/tests/module_spec_unittests.cc b/src/lib/config/tests/module_spec_unittests.cc
index 59f5459..1b43350 100644
--- a/src/lib/config/tests/module_spec_unittests.cc
+++ b/src/lib/config/tests/module_spec_unittests.cc
@@ -162,6 +162,10 @@ TEST(ModuleSpec, DataValidation) {
     EXPECT_FALSE(dataTest(dd, "data22_8.data"));
     EXPECT_FALSE(dataTest(dd, "data22_9.data"));
 
+    // Test if "version" is allowed in config data
+    // (same data as 22_7, but added "version")
+    EXPECT_TRUE(dataTest(dd, "data22_10.data"));
+
     ElementPtr errors = Element::createList();
     EXPECT_FALSE(dataTestWithErrors(dd, "data22_8.data", errors));
     EXPECT_EQ("[ \"Type mismatch\" ]", errors->str());