Skip to content

[YBState setStateChart:] isn't thread safe #2

Closed
nikolaykasyanov opened this Issue Jan 25, 2013 · 3 comments

2 participants

@nikolaykasyanov

You're using NSEnumerationConcurrent here, but it assumes concurrent block invocation, but [YBStatechart registerState:] modifies NSMutableDictionary, but NSMutableDictionary is not thread safe!

I haven't any problems on iOS, but on Simulator some assertions fails on completely correct state chart, I think this is due to thread unsafety.

@martijnthe
Collaborator
@nikolaykasyanov

Submitted pull request.
Thanks for nice library though.

@martijnthe
Collaborator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.