Skip to content

fix: Take into consideration when child is added to parent that is removed in the same tick#3428

Merged
spydon merged 4 commits intomainfrom
fix/enqueue-when-isremoving
Dec 30, 2024
Merged

fix: Take into consideration when child is added to parent that is removed in the same tick#3428
spydon merged 4 commits intomainfrom
fix/enqueue-when-isremoving

Conversation

@spydon
Copy link
Copy Markdown
Member

@spydon spydon commented Dec 30, 2024

Description

Currently we end up in a bad state if a child is added to a parent that is removed in the same tick, this PR fixes that by adding the child after the parent has been unmounted.

Closes: #3416

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

@spydon spydon merged commit 9a5c54b into main Dec 30, 2024
@spydon spydon deleted the fix/enqueue-when-isremoving branch December 30, 2024 23:42
nickf2k pushed a commit to nickf2k/flame that referenced this pull request Nov 16, 2025
…moved in the same tick (flame-engine#3428)

Currently we end up in a bad state if a child is added to a parent that
is removed in the same tick, this PR fixes that by adding the child
after the parent has been unmounted.

Closes: flame-engine#3416
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Unexpected concurrent modification

2 participants