- Notifications
You must be signed in to change notification settings - Fork20
Open
Description
Hi,
thanks for this neat little library! While exploring it, I was wondering why::tk/states is a sequence of maps with::tk/name rather than a map with::tk/name as keys. The latter would take care of preventing duplicate state definitions (which AFAICT don't serve a purpose and aren't detected by the validation function right now) as well as improve performance for large state machines because currently, each state lookup requires a linear search. I'm happy to provide a patch if you're interested. If so, let me know if you would like to retain backwards compatibility with the current API.
Metadata
Metadata
Assignees
Labels
No labels