Skip to content

Commit

Permalink
Fix comment phrasing to be more descriptive
Browse files Browse the repository at this point in the history
  • Loading branch information
tehampson committed Aug 16, 2022
1 parent 1f24589 commit 7790e7e
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions src/app/OperationalSessionSetup.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -125,10 +125,11 @@ void OperationalSessionSetup::Connect(Callback::Callback<OnDeviceConnected> * 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;
}

Expand Down

0 comments on commit 7790e7e

Please sign in to comment.