Skip to content

[TextServer] Fix range for zero-width glyphs extra spacing.#111964

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
bruvzg:zw_sp_1
Oct 23, 2025
Merged

[TextServer] Fix range for zero-width glyphs extra spacing.#111964
Repiteo merged 1 commit into
godotengine:masterfrom
bruvzg:zw_sp_1

Conversation

@bruvzg
Copy link
Copy Markdown
Member

@bruvzg bruvzg commented Oct 23, 2025

Fixes wrong range used #110317

Fixes #110301 for single character text.

@bruvzg bruvzg added this to the 4.6 milestone Oct 23, 2025
@bruvzg bruvzg requested a review from a team as a code owner October 23, 2025 18:41
@bruvzg bruvzg added bug regression topic:gui cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Oct 23, 2025
Copy link
Copy Markdown
Contributor

@Repiteo Repiteo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Repiteo Repiteo merged commit fec9ffa into godotengine:master Oct 23, 2025
20 checks passed
@Repiteo
Copy link
Copy Markdown
Contributor

Repiteo commented Oct 23, 2025

Thanks!

@akien-mga
Copy link
Copy Markdown
Member

Cherry-picked for 4.5.2.

@akien-mga akien-mga removed the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Jan 8, 2026
ArchercatNEO added a commit to ArchercatNEO/godot that referenced this pull request Mar 21, 2026
Fixes godotengine#117676
PR godotengine#111964 made a Ref<ColorManagementProfile> which we intentionally
leaked across a client-server barrier owned, which caused a use-after-free.
Manually adding reference/unreference calls keeps the ptr valid without memory leaks.
Sysex89 pushed a commit to Sysex89/godot that referenced this pull request Mar 26, 2026
Fixes godotengine#117676
PR godotengine#111964 made a Ref<ColorManagementProfile> which we intentionally
leaked across a client-server barrier owned, which caused a use-after-free.
Manually adding reference/unreference calls keeps the ptr valid without memory leaks.
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.

Having negative glyth extra spacing for FontVariations, breaks the minimum x size of the label

3 participants