BIND 10 trac553, updated. 500a785136096ec3a7c5259ec348a80256aed494 [trac553] Typos, spelling corrections, comments

BIND 10 source code commits bind10-changes at lists.isc.org
Thu Feb 17 19:41:32 UTC 2011


The branch, trac553 has been updated
       via  500a785136096ec3a7c5259ec348a80256aed494 (commit)
      from  ce7bc899e76309c63b1ad270b9474fdc1407cf5e (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 500a785136096ec3a7c5259ec348a80256aed494
Author: Michal 'vorner' Vaner <michal.vaner at nic.cz>
Date:   Thu Feb 17 20:40:38 2011 +0100

    [trac553] Typos, spelling corrections, comments

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

Summary of changes:
 src/lib/datasrc/memory_datasrc.cc                |    2 +-
 src/lib/datasrc/tests/memory_datasrc_unittest.cc |   22 +++++++++++-----------
 2 files changed, 12 insertions(+), 12 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/memory_datasrc.cc b/src/lib/datasrc/memory_datasrc.cc
index 20b60be..b2b49a7 100644
--- a/src/lib/datasrc/memory_datasrc.cc
+++ b/src/lib/datasrc/memory_datasrc.cc
@@ -449,7 +449,7 @@ struct MemoryZone::MemoryZoneImpl {
                      * have exactly one common label (the ".") in case we have
                      * nothing common under the node we got and we will get
                      * more common labels otherwise (yes, this relies on the
-                     * internal RBTree structure, which leaks out trough this
+                     * internal RBTree structure, which leaks out through this
                      * little bit).
                      *
                      * If the empty non-terminal node actually exists in the
diff --git a/src/lib/datasrc/tests/memory_datasrc_unittest.cc b/src/lib/datasrc/tests/memory_datasrc_unittest.cc
index 3f161ae..36fa104 100644
--- a/src/lib/datasrc/tests/memory_datasrc_unittest.cc
+++ b/src/lib/datasrc/tests/memory_datasrc_unittest.cc
@@ -302,7 +302,7 @@ public:
                     EXPECT_EQ(answer, find_result.rrset);
                 } else if (check_wild_answer) {
                     ASSERT_NE(ConstRRsetPtr(), answer) <<
-                        "Wrong test, don't check for wild names if you expect"
+                        "Wrong test, don't check for wild names if you expect "
                         "empty answer";
                     ASSERT_NE(ConstRRsetPtr(), find_result.rrset) <<
                         "No answer found";
@@ -330,7 +330,7 @@ public:
                 }
             });
     }
-    // Internal part of the cancelWildcard test that is run twice
+    // Internal part of the cancelWildcard test that is multiple times
     void doCancelWildcardTest();
 };
 
@@ -862,7 +862,7 @@ void
 MemoryZoneTest::doCancelWildcardTest() {
     // These should be canceled
     {
-        SCOPED_TRACE("Canceled under foo.example.org");
+        SCOPED_TRACE("Canceled under foo.wild.example.org");
         findTest(Name("aaa.foo.wild.example.org"), RRType::A(),
             Zone::NXDOMAIN);
         findTest(Name("zzz.foo.wild.example.org"), RRType::A(),
@@ -871,14 +871,14 @@ MemoryZoneTest::doCancelWildcardTest() {
 
     // This is existing, non-wildcard domain, shouldn't wildcard at all
     {
-        SCOPED_TRACE("Existing domain under foo.example.org");
+        SCOPED_TRACE("Existing domain under foo.wild.example.org");
         findTest(Name("bar.foo.wild.example.org"), RRType::A(), Zone::SUCCESS,
             true, rr_not_wild_);
     }
 
     // These should be caught by the wildcard
     {
-        SCOPED_TRACE("Neighbor wildcards to foo.example.org");
+        SCOPED_TRACE("Neighbor wildcards to foo.wild.example.org");
 
         const char* names[] = {
             "aaa.bbb.wild.example.org",
@@ -905,9 +905,9 @@ MemoryZoneTest::doCancelWildcardTest() {
 /*
  * This tests that if there's a name between the wildcard domain and the
  * searched one, it will not trigger wildcard, for example, if we have
- * *.wild.example.org and bar.foo.example.org, then we know foo.example.org
- * exists and is not wildcard. Therefore, search for aaa.foo.example.org should
- * return NXDOMAIN.
+ * *.wild.example.org and bar.foo.wild.example.org, then we know
+ * foo.wild.example.org exists and is not wildcard. Therefore, search for
+ * aaa.foo.wild.example.org should return NXDOMAIN.
  *
  * Tests few cases "around" the canceled wildcard match, to see something that
  * shouldn't be canceled isn't.
@@ -917,16 +917,16 @@ TEST_F(MemoryZoneTest, cancelWildcard) {
     EXPECT_EQ(SUCCESS, zone_.add(rr_not_wild_));
 
     {
-        SCOPED_TRACE("Runnig with single entry under foo.example.org");
+        SCOPED_TRACE("Runnig with single entry under foo.wild.example.org");
         doCancelWildcardTest();
     }
 
-    // Try putting another one undef foo.wild....
+    // Try putting another one under foo.wild....
     // The result should be the same but it will be done in another way in the
     // code, because the foo.wild.example.org will exist in the tree.
     EXPECT_EQ(SUCCESS, zone_.add(rr_not_wild_another_));
     {
-        SCOPED_TRACE("Runnig with two entries under foo.example.org");
+        SCOPED_TRACE("Runnig with two entries under foo.wild.example.org");
         doCancelWildcardTest();
     }
 }




More information about the bind10-changes mailing list