BIND 10 track1914, updated. 410df5ee446d69d53ca45e21307d07c54b5b2b76 [1914] a few fixes in comments

BIND 10 source code commits bind10-changes at lists.isc.org
Fri May 18 15:38:27 UTC 2012


The branch, track1914 has been updated
       via  410df5ee446d69d53ca45e21307d07c54b5b2b76 (commit)
      from  b3c7d1eaf3835c8bc13a7cc136b85e79aaf16934 (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 410df5ee446d69d53ca45e21307d07c54b5b2b76
Author: Jelte Jansen <jelte at isc.org>
Date:   Fri May 18 17:38:04 2012 +0200

    [1914] a few fixes in comments

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

Summary of changes:
 src/lib/config/ccsession.cc                 |    4 ++--
 src/lib/config/ccsession.h                  |    6 +++---
 src/lib/config/tests/ccsession_unittests.cc |    4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/config/ccsession.cc b/src/lib/config/ccsession.cc
index d968e14..a7ab9df 100644
--- a/src/lib/config/ccsession.cc
+++ b/src/lib/config/ccsession.cc
@@ -801,7 +801,7 @@ ModuleCCSession::checkAsyncRecv(const ConstElementPtr& envelope,
                                 const ConstElementPtr& msg)
 {
     for (AsyncRecvRequestID request(async_recv_requests_.begin());
-         request != async_recv_requests_.end(); ++ request) {
+         request != async_recv_requests_.end(); ++request) {
         // Just go through all the requests and look for a matching one
         if (requestMatch(*request, envelope)) {
             // We want the request to be still alive at the time we
@@ -814,7 +814,7 @@ ModuleCCSession::checkAsyncRecv(const ConstElementPtr& envelope,
                     requests_(requests),
                     request_(request)
                 { }
-                ~ RequestDeleter() {
+                ~RequestDeleter() {
                     requests_.erase(request_);
                 }
             private:
diff --git a/src/lib/config/ccsession.h b/src/lib/config/ccsession.h
index 25b65b7..e96a33d 100644
--- a/src/lib/config/ccsession.h
+++ b/src/lib/config/ccsession.h
@@ -411,17 +411,17 @@ public:
     /// are queued messages, the callback would be called once checkCommand()
     /// is invoked (possibly from start() or the constructor).
     ///
-    /// The matching is following. If is_reply is true, only replies are
+    /// The matching is as follows. If is_reply is true, only replies are
     /// considered. In that case, if seq is -1, any reply is accepted. If
     /// it is something else than -1, only the reply with matching seq is
-    /// taken. This may be used to receiving replies to commands
+    /// taken. This may be used to receive replies to commands
     /// asynchronously.
     ///
     /// In case the is_reply is false, the function looks for command messages.
     /// The seq parameter is ignored, but the recipient one is considered. If
     /// it is an empty string, any command is taken. If it is non-empty, only
     /// commands addressed to the recipient channel (eg. group - instance is
-    /// ignored for now)are taken. This can be used to receive foreign commands
+    /// ignored for now) are taken. This can be used to receive foreign commands
     /// or notifications. In such case, it might be desirable to call the
     /// groupRecvMsgAsync again from within the callback, to receive any future
     /// commands or events of the same type.
diff --git a/src/lib/config/tests/ccsession_unittests.cc b/src/lib/config/tests/ccsession_unittests.cc
index b4bc0b0..3fca741 100644
--- a/src/lib/config/tests/ccsession_unittests.cc
+++ b/src/lib/config/tests/ccsession_unittests.cc
@@ -713,7 +713,7 @@ TEST_F(CCSessionTest, doubleStartWithAddRemoteConfig) {
 
 /// \brief Test fixture for asynchronous receiving of messages.
 ///
-/// This is extension to the CCSessionTest. It would be possible to add
+/// This is an extension to the CCSessionTest. It would be possible to add
 /// the functionality to the CCSessionTest, but it is going to be used
 /// only by few tests and is non-trivial, so it is placed to a separate
 /// sub-class.
@@ -762,7 +762,7 @@ protected:
     void nothingCalled() {
         EXPECT_TRUE(called_.empty());
     }
-    /// \brief The testet session.
+    /// \brief The tested session.
     ModuleCCSession mccs_;
     /// \brief The value of message on the last called callback.
     ConstElementPtr last_msg_;



More information about the bind10-changes mailing list