Skip to content

Commit

Permalink
Clean up a couple of TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
tehampson committed Jun 8, 2022
1 parent a07d4c9 commit 0fbf786
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions src/app/clusters/door-lock-server/door-lock-server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,6 @@ class DoorLockClusterFabricDelegate : public chip::FabricTable::Delegate
{
void OnFabricHasChanged(FabricTable & fabricTable, FabricIndex fabricIndex, bool fabricDeleted) override
{
// TODO We likely want to do the same thing regardless of fabricDeleted. For
// now bailing out early when only update.
if (!fabricDeleted)
{
return;
Expand Down
2 changes: 0 additions & 2 deletions src/protocols/secure_channel/CASESession.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -160,8 +160,6 @@ void CASESession::InvalidateIfPendingEstablishment()
if (!IsSessionEstablishmentInProgress()) {
return;
}
// TODO Double check if there is maybe a more suitable CHIP_ERROR, some other options are
// CHIP_ERROR_CERT_EXPIRED, CHIP_ERROR_TRANSACTION_CANCELED, or CHIP_ERROR_INCORRECT_STATE
AbortPendingEstablish(CHIP_ERROR_CANCELLED);
}

Expand Down

0 comments on commit 0fbf786

Please sign in to comment.