Potential race condition in Store.run (?)

[stuartro]

In the code snippet below, Store's run method invokes transactionObj.run and then shortly (probably fast enough for it to work) invokes transactionObj.throttle. Would it not be better to reverse the order of run and throttle?

@discardableResult @inlinable @inline(__always)
public func run<A: ActionProtocol, M>(
    action: A,
    mode: Dispatcher.Strategy = .async(nil),
    throttle: TimeInterval = 0,
    handler: Dispatcher.TransactionCompletionHandler = nil
) -> Transaction<A> where A.AssociatedStoreType: Store<M> {
  let transactionObj = transaction(action: action, mode: mode)
  transactionObj.run(handler: handler) // RUN STARTED HERE
  if throttle > TimeInterval.ulpOfOne {
    transactionObj.throttle(throttle) // THROTTLING APPLIED HERE
  }
  return transactionObj
}

That is, instead of:

  let transactionObj = transaction(action: action, mode: mode)
  transactionObj.run(handler: handler) // RUN STARTED HERE
  if throttle > TimeInterval.ulpOfOne {
    transactionObj.throttle(throttle) // THROTTLING APPLIED HERE
  }

use:

  let transactionObj = transaction(action: action, mode: mode)

  if throttle > TimeInterval.ulpOfOne {
    transactionObj.throttle(throttle) // THROTTLING APPLIED HERE
  }
  transactionObj.run(handler: handler) // RUN STARTED HERE

?

[alexdrone]

Good point :)