Fix for 22822 ComputedStates example

[scottgshannon]

Objective

• Fixes computed_states example broken #22822

Solution

• Adds ALLOW_SAME_STATE_TRANSITIONS = false to ComputedStates in computed_states example

Testing

• Tested by running example on x86_64 Linux 4.19 + KDE. Behavior noted in computed_states example broken #22822 no longer occurs.

[github-actions]

Welcome, new contributor!

Please make sure you've read our contributing guide, as well as our policy regarding AI usage, and we look forward to reviewing your pull request shortly ✨

[kfc35]

If you verify that you actually don’t need this line:

Suggested change

const ALLOW_SAME_STATE_TRANSITIONS: bool = false;

[scottgshannon]

TurboMode can be re-entered (via activating turbo then pausing). This produces duplicate UI elements that overlap and thus don't cause visual artifacts.

[kfc35]

Ah I see, I overlooked that, thank you for pointing that out

[kfc35]

If you verify that you actually don’t need this line:

Suggested change

const ALLOW_SAME_STATE_TRANSITIONS: bool = false;

[scottgshannon]

While IsPaused cannot be re-entered it is tied to an OnEnter system that performs command.spawn calls like the other computed states. This line could be removed and retain proper behavior but I believe the example is stronger by keeping it.

[kfc35]

Thanks for your first contribution!