From 7790e7e088a206efdc898421e094c11b48640e5e Mon Sep 17 00:00:00 2001 From: Terence Hampson Date: Tue, 16 Aug 2022 13:10:33 +0000 Subject: [PATCH] Fix comment phrasing to be more descriptive --- src/app/OperationalSessionSetup.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/app/OperationalSessionSetup.cpp b/src/app/OperationalSessionSetup.cpp index 5917a4c4b9bcdf..3a0d124297a091 100644 --- a/src/app/OperationalSessionSetup.cpp +++ b/src/app/OperationalSessionSetup.cpp @@ -125,10 +125,11 @@ void OperationalSessionSetup::Connect(Callback::Callback * on isConnected = AttachToExistingSecureSession(); if (!isConnected) { - // We should really be in State::HasAddress since in the same call - // we move to State::HasAddress we then moved to State::Connecting - // or call DequeueConnectionCallbacks with an error thus releasing - // ourselve. + // We should not actually every be in be in State::HasAddress. This + // is because in the same call that we moved to State::HasAddress + // we either move to State::Connecting or call + // DequeueConnectionCallbacks with an error thus releasing + // ourselves before any call would reach this section of code. err = CHIP_ERROR_INCORRECT_STATE; }