Merge pull request #41 from lucabartoletti/fix-try-warnings-after-throws-removal

Remove unnecessary try from primaryKeyValues() call sites

Author: marcoarment

Sources/Blackbird/BlackbirdCache.swift

@@ -376,13 +376,13 @@ extension Blackbird.Database {
 extension BlackbirdModel {
     internal func _saveCachedInstance(for database: Blackbird.Database) {
         let cacheLimit = Self.cacheLimit
-        if cacheLimit > 0, let pkValues = try? self.primaryKeyValues(), pkValues.count == 1, let pk = try? Blackbird.Value.fromAny(pkValues.first!) {
+        if cacheLimit > 0, case let pkValues = self.primaryKeyValues(), pkValues.count == 1, let pk = try? Blackbird.Value.fromAny(pkValues.first!) {
             database.cache.writeModel(tableName: Self.tableName, primaryKey: pk, instance: self, entryLimit: cacheLimit)
         }
     }
 
     internal func _deleteCachedInstance(for database: Blackbird.Database) {
-        if Self.cacheLimit > 0, let pkValues = try? self.primaryKeyValues(), pkValues.count == 1, let pk = try? Blackbird.Value.fromAny(pkValues.first!) {
+        if Self.cacheLimit > 0, case let pkValues = self.primaryKeyValues(), pkValues.count == 1, let pk = try? Blackbird.Value.fromAny(pkValues.first!) {
             database.cache.deleteModel(tableName: Self.tableName, primaryKey: pk)
         }
     }

Sources/Blackbird/BlackbirdObservation.swift

@@ -118,7 +118,7 @@ public final class BlackbirdModelQueryObserver<T: BlackbirdModel, R: Sendable> {
 extension BlackbirdModel {
     public typealias Observer = BlackbirdModelObserver<Self>
 
-    public var observer: Observer { Observer(multicolumnPrimaryKey: try! primaryKeyValues()) }
+    public var observer: Observer { Observer(multicolumnPrimaryKey: primaryKeyValues()) }
 }
 
 @available(macOS 14, iOS 17, tvOS 17, watchOS 10, *)

Sources/Blackbird/BlackbirdSwiftUI.swift

@@ -199,7 +199,7 @@ extension Blackbird {
 
     public init(_ instance: T, updatesEnabled: Bool = true) {
         _instance = State(initialValue: instance)
-        instanceObserver = BlackbirdModelInstanceChangeObserver<T>(primaryKeyValues: try! instance.primaryKeyValues().map { try! Blackbird.Value.fromAny($0) })
+        instanceObserver = BlackbirdModelInstanceChangeObserver<T>(primaryKeyValues: instance.primaryKeyValues().map { try! Blackbird.Value.fromAny($0) })
         instanceObserver.updatesEnabled = updatesEnabled
     }