Merge pull request #5 from shapehq/fix/hans-shape/main_actor_spice_dispenser

Hans-shape · web-flow · commit d1d506ac1a28 · 2024-06-19T10:57:24.000+02:00
Make some properties of SpiceDispenser MainActor
diff --git a/SHPSpices/Source/Spice/SpiceDispenser.swift b/SHPSpices/Source/Spice/SpiceDispenser.swift
@@ -13,8 +13,8 @@ enum SpiceDispenserProperty {
     case spice(String, SpiceType)
 }
 
-public protocol SpiceDispenser: class {
-    var store: UserDefaults { get }
+public protocol SpiceDispenser: AnyObject {
+    @MainActor var store: UserDefaults { get }
     var title: String? { get }
 }
 
@@ -29,11 +29,13 @@ public extension SpiceDispenser {
         return .standard
     }
     
+    @MainActor
     func prepare(with application: UIApplication? = nil) {
         recursivePrepare(with: application, rootSpiceDispenser: self, path: [])
         validateValues()
     }
     
+    @MainActor
     internal func recursivePrepare(with application: UIApplication?, rootSpiceDispenser: SpiceDispenser, path: [String]) {
         properties().forEach { property in
             switch property {

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ enum SpiceDispenserProperty {`
`13`	`13`	`case spice(String, SpiceType)`
`14`	`14`	`}`
`15`	`15`
`16`		`-public protocol SpiceDispenser: class {`
`17`		`- var store: UserDefaults { get }`
	`16`	`+public protocol SpiceDispenser: AnyObject {`
	`17`	`+ @MainActor var store: UserDefaults { get }`
`18`	`18`	`var title: String? { get }`
`19`	`19`	`}`
`20`	`20`
`@@ -29,11 +29,13 @@ public extension SpiceDispenser {`
`29`	`29`	`return .standard`
`30`	`30`	`}`
`31`	`31`
	`32`	`+ @MainActor`
`32`	`33`	`func prepare(with application: UIApplication? = nil) {`
`33`	`34`	`recursivePrepare(with: application, rootSpiceDispenser: self, path: [])`
`34`	`35`	`validateValues()`
`35`	`36`	`}`
`36`	`37`
	`38`	`+ @MainActor`
`37`	`39`	`internal func recursivePrepare(with application: UIApplication?, rootSpiceDispenser: SpiceDispenser, path: [String]) {`
`38`	`40`	`properties().forEach { property in`
`39`	`41`	`switch property {`