Skip to content

Commit cd5545b

Browse files
authored
fix(datastore): memory leak in ModelSyncedEventEmitter (#3675)
* fix(datastore): memory leak in ModelSyncedEventEmitter * resolve lint comments
1 parent dd64b90 commit cd5545b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

AmplifyPlugins/DataStore/Sources/AWSDataStorePlugin/Sync/InitialSync/ModelSyncedEventEmitter.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ final class ModelSyncedEventEmitter {
6565
self.syncOrchestratorSink = initialSyncOrchestrator?
6666
.publisher
6767
.receive(on: queue)
68-
.filter(filterSyncOperationEvent(_:))
68+
.filter { [weak self] in self?.filterSyncOperationEvent($0) == true }
6969
.sink(receiveCompletion: { _ in },
7070
receiveValue: { [weak self] value in
7171
self?.onReceiveSyncOperationEvent(value: value)
@@ -74,7 +74,7 @@ final class ModelSyncedEventEmitter {
7474
self.reconciliationQueueSink = reconciliationQueue?
7575
.publisher
7676
.receive(on: queue)
77-
.filter(filterReconciliationQueueEvent(_:))
77+
.filter { [weak self] in self?.filterReconciliationQueueEvent($0) == true }
7878
.sink(receiveCompletion: { _ in },
7979
receiveValue: { [weak self] value in
8080
self?.onReceiveReconciliationEvent(value: value)

0 commit comments

Comments
 (0)