Skip to content

Commit

Permalink
remove unnecessary checks
Browse files Browse the repository at this point in the history
  • Loading branch information
mkardous-silabs committed Jan 19, 2024
1 parent 7497fd6 commit b582fab
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/app/icd/ICDManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -468,14 +468,11 @@ void ICDManager::OnActiveRequestWithdrawal(KeepActiveFlags request)

void ICDManager::OnNetworkActivity()
{
assertChipStackLockedByCurrentThread();
this->UpdateOperationState(OperationalState::ActiveMode);
}

void ICDManager::OnICDManagementServerEvent(ICDManagementEvents event)
{
assertChipStackLockedByCurrentThread();

switch (event)
{
case ICDManagementEvents::kTableUpdated:
Expand All @@ -493,8 +490,6 @@ void ICDManager::OnICDManagementServerEvent(ICDManagementEvents event)

void ICDManager::OnSubscriptionReport()
{
assertChipStackLockedByCurrentThread();

// If the device is already in ActiveMode, that means that all active subscriptions have already been marked dirty.
// Since we only mark them dirty when we enter ActiveMode, it is not necessary to update the operational state a second time.
// Doing so will only add an ActiveModeThreshold to the active time which we don't want to do here.
Expand Down

0 comments on commit b582fab

Please sign in to comment.