Skip to content

Commit b04ec4f

Browse files
authored
Internal state is updated too often, removing this noise (#36056)
1 parent 650e69b commit b04ec4f

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

src/darwin/Framework/CHIP/MTRDevice_Concrete.mm

+2-8
Original file line numberDiff line numberDiff line change
@@ -772,8 +772,6 @@ - (void)_ensureSubscriptionForExistingDelegates:(NSString *)reason
772772
} errorHandler:nil];
773773
}
774774
}
775-
776-
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
777775
}
778776

779777
- (void)invalidate
@@ -1247,7 +1245,6 @@ - (void)_handleResubscriptionNeededWithDelayOnDeviceQueue:(NSNumber *)resubscrip
12471245
std::lock_guard lock(_descriptionLock);
12481246
_lastSubscriptionFailureTimeForDescription = _lastSubscriptionFailureTime;
12491247
}
1250-
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
12511248
BOOL deviceUsesThread = [self _deviceUsesThread];
12521249

12531250
// If a previous resubscription failed, remove the item from the subscription pool.
@@ -1325,7 +1322,6 @@ - (void)_doHandleSubscriptionReset:(NSNumber * _Nullable)retryDelay
13251322
std::lock_guard lock(_descriptionLock);
13261323
_lastSubscriptionFailureTimeForDescription = _lastSubscriptionFailureTime;
13271324
}
1328-
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
13291325

13301326
// if there is no delegate then also do not retry
13311327
if (![self _delegateExists]) {
@@ -1700,8 +1696,6 @@ - (void)_scheduleClusterDataPersistence
17001696
}
17011697
}
17021698

1703-
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
1704-
17051699
// Do not schedule persistence if device is reporting excessively
17061700
if ([self _deviceIsReportingExcessively]) {
17071701
return;
@@ -1767,6 +1761,7 @@ - (void)_handleReportEnd
17671761
[delegate deviceConfigurationChanged:self];
17681762
}
17691763
}];
1764+
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
17701765
_deviceConfigurationChanged = NO;
17711766
}
17721767

@@ -1781,6 +1776,7 @@ - (void)_handleReportEnd
17811776
// all the data for the device now.
17821777
_deviceCachePrimed = YES;
17831778
[self _callDelegateDeviceCachePrimed];
1779+
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
17841780
}
17851781

17861782
// For unit testing only
@@ -1791,8 +1787,6 @@ - (void)_handleReportEnd
17911787
}
17921788
}];
17931789
#endif
1794-
1795-
[self _notifyDelegateOfPrivateInternalPropertiesChanges];
17961790
}
17971791

17981792
- (BOOL)_interestedPaths:(NSArray * _Nullable)interestedPaths includesAttributePath:(MTRAttributePath *)attributePath

0 commit comments

Comments
 (0)