Commit 410df5ee authored by Jelte Jansen's avatar Jelte Jansen
Browse files

[1914] a few fixes in comments

parent b3c7d1ea
......@@ -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:
......
......@@ -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.
......
......@@ -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_;
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment