From 0fbf786543eaa8f2583810541d068bf78633af7f Mon Sep 17 00:00:00 2001 From: Terence Hampson Date: Wed, 8 Jun 2022 19:15:24 +0000 Subject: [PATCH] Clean up a couple of TODOs --- src/app/clusters/door-lock-server/door-lock-server.cpp | 2 -- src/protocols/secure_channel/CASESession.cpp | 2 -- 2 files changed, 4 deletions(-) diff --git a/src/app/clusters/door-lock-server/door-lock-server.cpp b/src/app/clusters/door-lock-server/door-lock-server.cpp index db4dcfb5aa8973..f9318e44516dee 100644 --- a/src/app/clusters/door-lock-server/door-lock-server.cpp +++ b/src/app/clusters/door-lock-server/door-lock-server.cpp @@ -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; diff --git a/src/protocols/secure_channel/CASESession.cpp b/src/protocols/secure_channel/CASESession.cpp index 40391fd0f64bf7..f34b0504939414 100644 --- a/src/protocols/secure_channel/CASESession.cpp +++ b/src/protocols/secure_channel/CASESession.cpp @@ -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); }