Skip to content

Fix potential crash during resource reloading#8500

Merged
4ian merged 2 commits intomasterfrom
claude/fix-spine-preview-flashing-jNbX8
Apr 9, 2026
Merged

Fix potential crash during resource reloading#8500
4ian merged 2 commits intomasterfrom
claude/fix-spine-preview-flashing-jNbX8

Conversation

@4ian
Copy link
Copy Markdown
Owner

@4ian 4ian commented Apr 9, 2026

No description provided.

claude added 2 commits April 9, 2026 10:50
Use unique pause reasons for each _reloadResources call to prevent a
race condition. _renderingPausedReasons is a Set, so adding the same
'resource-reload' reason twice was a no-op, but removing it once
removed it completely. When two reloads overlapped (e.g. from
forceUpdateRenderedInstancesOfObject and onResourceExternallyChanged),
the first to complete would resume rendering while the second was still
unloading textures, causing "Cannot read properties of null (reading
'_batchEnabled')" crashes on Spine objects.

https://claude.ai/code/session_01F2PXHKPyT9aASm5q3r8CWh
Resize can run during a React prop update while _reloadResources is
mid-flight (textures disposed). Interacting with PixiJS/Three.js state
in that window could crash. Without try/catch, this would be an
unhandled exception bubbling to React's error boundary. A failed resize
is recoverable on the next frame.

https://claude.ai/code/session_01F2PXHKPyT9aASm5q3r8CWh
@4ian 4ian merged commit 87bfe3e into master Apr 9, 2026
6 checks passed
@4ian 4ian deleted the claude/fix-spine-preview-flashing-jNbX8 branch April 9, 2026 13:50
NBForgeLab added a commit to NBForgeLab/GDevelop that referenced this pull request Apr 18, 2026
* Make the home page navigation sidebar collapsible

* Fix potential crashes in the scene editor

* Improve logs in case of UseAfterFreeError

Only show in developer changelog

* Fix potential crashes in the scene editor

Don't show in changelog

* Display extension details side by side with the extension list on large screens (4ian#8482)

* Fix hot-reloading of global object instances (4ian#8489)

* Fix "Add or edit variables" button always creating variables for object variables (4ian#8493)

* Fix "Add or edit variables" button opening scene variables instead of local variables (4ian#8496)

* Add save/restore support for linked object relationships (4ian#8499)

Fix 4ian#8497

* Fix potential crash during resource reloading (4ian#8500)

* Make clear in Storage actions if a number or text is being written (4ian#8498)

* Add "Reload Project" menu action (4ian#8501)

Co-authored-by: Gleb Volkov <glebusheg@playtika.com>

* Remove deprecated, unused GitHub workflow

Don't show in changelog

* Fix scene properties not usable via the variable action and condition (4ian#8504)

* [Auto PR] Update translations (4ian#8474)

* Bump newIDE version

* Add experimental option in tabs context menu to open some editors (Event Sheets, Extensions...) into separate windows (4ian#8492)

* Fix wavesurfer version (4ian#8508)

* Allow AI to create objects via exact asset ID without specifying type (4ian#8509)

* Fix crashes when using invalid object names in object folders

* Support partial asset ID matching in asset search (4ian#8512)

Don't show in changelog

* Fix missing globally used resources when unloading a scene that uses these resources (4ian#8513)

- It only happens when "Unload at scene exit" is selected in project settings and a global object share some resources with a scene object.

* Fix tentatively crash in popped-out window when a theme is missing a color (4ian#8518)

Fix 4ian#8517

---------

Co-authored-by: Florian Rival <Florian.rival@gmail.com>
Co-authored-by: D8H <Davy.Helard@gmail.com>
Co-authored-by: LuniMoon <103995399+LuniMoon@users.noreply.github.com>
Co-authored-by: Gleb Volkov <glebusheg@playtika.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
NBForgeLab added a commit to NBForgeLab/GDevelop that referenced this pull request Apr 18, 2026
* Make the home page navigation sidebar collapsible

* Fix potential crashes in the scene editor

* Improve logs in case of UseAfterFreeError

Only show in developer changelog

* Fix potential crashes in the scene editor

Don't show in changelog

* Display extension details side by side with the extension list on large screens (4ian#8482)

* Fix hot-reloading of global object instances (4ian#8489)

* Fix "Add or edit variables" button always creating variables for object variables (4ian#8493)

* Fix "Add or edit variables" button opening scene variables instead of local variables (4ian#8496)

* Add save/restore support for linked object relationships (4ian#8499)

Fix 4ian#8497

* Fix potential crash during resource reloading (4ian#8500)

* Make clear in Storage actions if a number or text is being written (4ian#8498)

* Add "Reload Project" menu action (4ian#8501)



* Remove deprecated, unused GitHub workflow

Don't show in changelog

* Fix scene properties not usable via the variable action and condition (4ian#8504)

* [Auto PR] Update translations (4ian#8474)

* Bump newIDE version

* Add experimental option in tabs context menu to open some editors (Event Sheets, Extensions...) into separate windows (4ian#8492)

* Fix wavesurfer version (4ian#8508)

* Allow AI to create objects via exact asset ID without specifying type (4ian#8509)

* Fix crashes when using invalid object names in object folders

* Support partial asset ID matching in asset search (4ian#8512)

Don't show in changelog

* Fix missing globally used resources when unloading a scene that uses these resources (4ian#8513)

- It only happens when "Unload at scene exit" is selected in project settings and a global object share some resources with a scene object.

* Fix tentatively crash in popped-out window when a theme is missing a color (4ian#8518)

Fix 4ian#8517

---------

Co-authored-by: Florian Rival <Florian.rival@gmail.com>
Co-authored-by: D8H <Davy.Helard@gmail.com>
Co-authored-by: LuniMoon <103995399+LuniMoon@users.noreply.github.com>
Co-authored-by: Gleb Volkov <glebusheg@playtika.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants