remove lock instance

Ryu0118 · Oct 1, 2023 · 5e0ed52 · 5e0ed52
1 parent 152874e
commit 5e0ed52
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 10 deletions.
diff --git a/Sources/SimplexArchitecture/StateContainer.swift b/Sources/SimplexArchitecture/StateContainer.swift
@@ -1,8 +1,7 @@
 import Foundation
 import XCTestDynamicOverlay
 
-/// StateContainer is not thread-safe. Therefore, StateContainer must use NSLock or NSRecursiveLock for exclusions when changing values.
-/// In Store, NSRecursiveLock is used for exclusions when executing the `reduce(into:action)`.
+/// StateContainer is not thread-safe. StateContainer must be accessed from MainActor.
 @dynamicMemberLookup
 public final class StateContainer<Target: ActionSendable> {
     public var reducerState: Target.Reducer.ReducerState {

diff --git a/Sources/SimplexArchitecture/Store/Store+send.swift b/Sources/SimplexArchitecture/Store/Store+send.swift
@@ -118,13 +118,6 @@ extension Store {
         }
     }
 
-    @inlinable
-    func withLock<T>(_ operation: () throws -> T) rethrows -> T {
-        lock.lock()
-        defer { lock.unlock() }
-        return try operation()
-    }
-
     func makeSend(for container: StateContainer<Reducer.Target>) -> Send<Reducer> {
         Send(
             sendAction: { [weak self] action in

diff --git a/Sources/SimplexArchitecture/Store/Store.swift b/Sources/SimplexArchitecture/Store/Store.swift
@@ -16,7 +16,6 @@ public final class Store<Reducer: ReducerProtocol> {
     // Buffer to store Actions recurrently invoked through SideEffect in a single Action sent from View
     @TestOnly var sentFromEffectActions: [ActionTransition<Reducer>] = []
 
-    @usableFromInline let lock = NSRecursiveLock()
     @usableFromInline var pullbackAction: ((Reducer.Action) -> Void)?
     @usableFromInline var pullbackReducerAction: ((Reducer.ReducerAction) -> Void)?