From 75e18897540b624e6de2cf23d39ec84c407b329f Mon Sep 17 00:00:00 2001 From: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> Date: Wed, 11 Oct 2023 13:33:00 -0400 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Boris Zbarsky --- src/messaging/ExchangeContext.h | 2 +- src/messaging/tests/TestReliableMessageProtocol.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/messaging/ExchangeContext.h b/src/messaging/ExchangeContext.h index 5f925af2c16642..fc20a5aace5273 100644 --- a/src/messaging/ExchangeContext.h +++ b/src/messaging/ExchangeContext.h @@ -215,7 +215,7 @@ class DLL_EXPORT ExchangeContext : public ReliableMessageContext, bool IsSendExpected() const { return mFlags.Has(Flags::kFlagWillSendMessage); } /** - * Tracks if we have received at least one application level message + * Tracks whether we have received at least one application level message * during the life-time of this exchange * * @return Returns 'true' if we have received at least one message, else 'false' diff --git a/src/messaging/tests/TestReliableMessageProtocol.cpp b/src/messaging/tests/TestReliableMessageProtocol.cpp index 2bc643fbc47f98..325f27c782832c 100644 --- a/src/messaging/tests/TestReliableMessageProtocol.cpp +++ b/src/messaging/tests/TestReliableMessageProtocol.cpp @@ -1574,7 +1574,7 @@ void TestReliableMessageProtocol::CheckIsPeerActiveNotInitiator(nlTestSuite * in { /** * This tests the following scenario: - * 1) A reliable message expecting a response is sent from the initiator to responder which is losts + * 1) A reliable message expecting a response is sent from the initiator to responder which is lost * 2) Initiator resends the message at the IdleRetrans interval * 3) Responder receives the message and sends a standalone ack * 4) Responder sends a response and fails