From: Jelte Jansen Date: Fri, 18 May 2012 15:38:04 +0000 (+0200) Subject: [1914] a few fixes in comments X-Git-Tag: trac2351_base~226^2~89^2~1 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=410df5ee446d69d53ca45e21307d07c54b5b2b76;p=thirdparty%2Fkea.git [1914] a few fixes in comments --- diff --git a/src/lib/config/ccsession.cc b/src/lib/config/ccsession.cc index d968e143c3..a7ab9df797 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 25b65b7509..e96a33d44b 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 b4bc0b08a2..3fca741bbb 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_;