Skip to content

[shaping] Make explicit non-mark category take precedence over ad-hoc mark detection#2508

Merged
justvanrossum merged 1 commit intomainfrom
fix-ad-hoc-mark-detection-issue-2507
Mar 23, 2026
Merged

[shaping] Make explicit non-mark category take precedence over ad-hoc mark detection#2508
justvanrossum merged 1 commit intomainfrom
fix-ad-hoc-mark-detection-issue-2507

Conversation

@justvanrossum
Copy link
Copy Markdown
Member

If a glyph has an explicit category, either custom (via glyph infos) or via the Glyph Data list, and that category/subcategory is not Mark/Nonspacing, then the glyph is definitely not a mark, regardless of what the ad-hoc mark detection heuristic may think.

This fixes #2507.

@justvanrossum justvanrossum reopened this Mar 23, 2026
@justvanrossum justvanrossum marked this pull request as ready for review March 23, 2026 08:46
@justvanrossum justvanrossum merged commit 99e3273 into main Mar 23, 2026
8 checks passed
@justvanrossum justvanrossum deleted the fix-ad-hoc-mark-detection-issue-2507 branch March 23, 2026 08:53
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.

Some base glyph advances seem to incorrectly zeroed

1 participant