Skip to content

Commit

Permalink
Fix typos in README and remove commented code
Browse files Browse the repository at this point in the history
  • Loading branch information
vadymmarkov committed Aug 7, 2018
1 parent d1faa04 commit 1c0e873
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 18 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ removed automatically when the weekly captured observer has been deallocated.

```swift
// Add observer
let token = storage.addStorageObserver(self) { observer, storage, change
let token = storage.addStorageObserver(self) { observer, storage, change in
switch change {
case .add(let key):
print("Added \(key)")
Expand Down Expand Up @@ -356,7 +356,7 @@ let token = storage.addObserver(self, forKey: key) { observer, storage, change i
token.cancel()

// Remove observer for key
storage.removeObserver(forKey: "user1")
storage.removeObserver(forKey: key)

// Remove all observers
storage.removeAllKeyObservers()
Expand Down
4 changes: 2 additions & 2 deletions Source/Shared/Storage/HybridStorage.swift
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ public final class HybridStorage<T> {
public let memoryStorage: MemoryStorage<T>
public let diskStorage: DiskStorage<T>

private(set) var storageObservations = [UUID : (HybridStorage, StorageChange) -> Void]()
private(set) var keyObservations = [String : (HybridStorage, KeyChange<T>) -> Void]()
private(set) var storageObservations = [UUID: (HybridStorage, StorageChange) -> Void]()
private(set) var keyObservations = [String: (HybridStorage, KeyChange<T>) -> Void]()

public init(memoryStorage: MemoryStorage<T>, diskStorage: DiskStorage<T>) {
self.memoryStorage = memoryStorage
Expand Down
14 changes: 0 additions & 14 deletions Source/Shared/Storage/Storage.swift
Original file line number Diff line number Diff line change
Expand Up @@ -36,24 +36,10 @@ public final class Storage<T> {
storage: hybridStorage,
serialQueue: DispatchQueue(label: "Cache.AsyncStorage.SerialQueue")
)
subscribeToChanges(in: hybridStorage)
}

/// Used for async operations
public lazy var async = self.asyncStorage

private func subscribeToChanges(in storage: HybridStorage<T>) {
// storage.storageObservationRegistry.addObservation { [weak self] _, change in
// guard let strongSelf = self else { return }
// strongSelf.storageObservationRegistry.notifyObservers(about: change, in: strongSelf)
// }
// keyObservationRegistry.onNewKey = { [weak self] key in
// guard let strongSelf = self else { return }
// storage.keyObservationRegistry.addObservation({ _, change in
// strongSelf.keyObservationRegistry.notifyObserver(forKey: key, about: change, in: strongSelf)
// }, forKey: key)
// }
}
}

extension Storage: StorageAware {
Expand Down

0 comments on commit 1c0e873

Please sign in to comment.