Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Added an AddTimedTransition() method #5

Closed
wants to merge 2 commits into from

Conversation

mrisher
Copy link

@mrisher mrisher commented Jul 24, 2022

To simplify transitions based on the state's MAXTIME field and avoid the complexity involved in lambda functions inside the AddTransition call, I created an AddTimedTransition() method which performs the _currentState->timeout boolean check inside Update().

For clarity, this adds a bool TimedTransition flag (and thus, one byte per transition) inside the FSM_Transition structure; if that is wasteful, we could probably overload Condition and ConditionVar (maybe if they are both null?.

Thank you for considering!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants