Address review comment.

bzbarsky-apple · bzbarsky-apple · commit 20a4394ceffc · 2024-05-09T12:09:47.000-04:00
diff --git a/src/messaging/ExchangeMgr.cpp b/src/messaging/ExchangeMgr.cpp
@@ -107,12 +107,12 @@ ExchangeContext * ExchangeManager::NewContext(const SessionHandle & session, Exc
     if (!session->IsActiveSession())
     {
 #if CHIP_ERROR_LOGGING
-        // Disallow creating exchange on an inactive session
         const ScopedNodeId & peer = session->GetPeer();
         ChipLogError(ExchangeManager, "NewContext failed: session %u to " ChipLogFormatScopedNodeId " is inactive",
                      session->SessionIdForLogging(), ChipLogValueScopedNodeId(peer));
 #endif // CHIP_ERROR_LOGGING
 
+        // Disallow creating exchange on an inactive session
         return nullptr;
     }
     return mContextPool.CreateObject(this, mNextExchangeId++, session, isInitiator, delegate);

Original file line number	Diff line number	Diff line change
`@@ -107,12 +107,12 @@ ExchangeContext * ExchangeManager::NewContext(const SessionHandle & session, Exc`
`107`	`107`	`if (!session->IsActiveSession())`
`108`	`108`	`{`
`109`	`109`	`#if CHIP_ERROR_LOGGING`
`110`		`- // Disallow creating exchange on an inactive session`
`111`	`110`	`const ScopedNodeId & peer = session->GetPeer();`
`112`	`111`	`ChipLogError(ExchangeManager, "NewContext failed: session %u to " ChipLogFormatScopedNodeId " is inactive",`
`113`	`112`	`session->SessionIdForLogging(), ChipLogValueScopedNodeId(peer));`
`114`	`113`	`#endif // CHIP_ERROR_LOGGING`
`115`	`114`
	`115`	`+ // Disallow creating exchange on an inactive session`
`116`	`116`	`return nullptr;`
`117`	`117`	`}`
`118`	`118`	`return mContextPool.CreateObject(this, mNextExchangeId++, session, isInitiator, delegate);`