Compare commits
No commits in common. "f90c48eb810a57bec191cff7d5c336545ba395ce" and "423c4ce7a42a775baaff11f20ead7947901ef0bd" have entirely different histories.
f90c48eb81
...
423c4ce7a4
|
@ -265,46 +265,22 @@ void LimboHSM::_validate_property(PropertyInfo &p_property) const {
|
|||
}
|
||||
}
|
||||
|
||||
void LimboHSM::_exit_if_not_inside_tree() {
|
||||
if (is_active() && !is_inside_tree()) {
|
||||
_exit();
|
||||
}
|
||||
}
|
||||
|
||||
void LimboHSM::_notification(int p_what) {
|
||||
switch (p_what) {
|
||||
case NOTIFICATION_POST_ENTER_TREE: {
|
||||
if (was_active && is_root()) {
|
||||
// Re-activate the root HSM if it was previously active.
|
||||
// Typically, this happens when the node is re-entered scene repeatedly (such as with object pooling).
|
||||
// Typically, this happens when the node is re-entered scene repeatedly (e.g., re-parenting, pooling).
|
||||
set_active(true);
|
||||
}
|
||||
} break;
|
||||
case NOTIFICATION_EXIT_TREE: {
|
||||
if (is_root()) {
|
||||
// Exit the state machine if the root HSM is no longer in the scene tree (except when being reparented).
|
||||
// This ensures that resources and signal connections are released if active.
|
||||
was_active = is_active();
|
||||
// Remember active status for re-parenting and exit state machine
|
||||
// to release resources and signal connections if active.
|
||||
was_active = active;
|
||||
if (is_active()) {
|
||||
// Check if the HSM node is being deleted.
|
||||
bool is_being_deleted = false;
|
||||
Node *node = this;
|
||||
while (node) {
|
||||
if (node->is_queued_for_deletion()) {
|
||||
is_being_deleted = true;
|
||||
break;
|
||||
}
|
||||
node = node->get_parent();
|
||||
}
|
||||
|
||||
if (is_being_deleted) {
|
||||
// Exit the state machine immediately if the HSM is being deleted.
|
||||
_exit();
|
||||
} else {
|
||||
// Use deferred mode to prevent exiting during Node re-parenting.
|
||||
// This allows the HSM to remain active when it (or one of its parents) is reparented.
|
||||
callable_mp(this, &LimboHSM::_exit_if_not_inside_tree).call_deferred();
|
||||
}
|
||||
_exit();
|
||||
}
|
||||
}
|
||||
} break;
|
||||
|
|
|
@ -63,7 +63,6 @@ private:
|
|||
HashMap<TransitionKey, Transition, TransitionKeyHasher> transitions;
|
||||
|
||||
void _get_transition(LimboState *p_from_state, const StringName &p_event, Transition &r_transition) const;
|
||||
void _exit_if_not_inside_tree();
|
||||
|
||||
protected:
|
||||
static void _bind_methods();
|
||||
|
|
Loading…
Reference in New Issue