X Tutup
Skip to content

Fix memory management for ColorPalette in save file dialog#113573

Merged
akien-mga merged 1 commit intogodotengine:masterfrom
MattiaZir:fix-color-picker-crash
Dec 5, 2025
Merged

Fix memory management for ColorPalette in save file dialog#113573
akien-mga merged 1 commit intogodotengine:masterfrom
MattiaZir:fix-color-picker-crash

Conversation

@MattiaZir
Copy link
Contributor

Using Ref<T> instead of creating a raw pointer seems to have fixed #113556.

@KoBeWi
Copy link
Member

KoBeWi commented Dec 4, 2025

The commits need to be squashed.

@MattiaZir MattiaZir force-pushed the fix-color-picker-crash branch from 23fc647 to 7d77641 Compare December 4, 2025 16:36
@MattiaZir MattiaZir force-pushed the fix-color-picker-crash branch from 7d77641 to bf85f96 Compare December 4, 2025 16:39
@MattiaZir
Copy link
Contributor Author

Done, thank you again!

@akien-mga akien-mga merged commit 8bff45a into godotengine:master Dec 5, 2025
20 checks passed
@akien-mga
Copy link
Member

Thanks!

@MattiaZir MattiaZir deleted the fix-color-picker-crash branch December 5, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash when saving color palettes from ColorPicker

4 participants

X Tutup