Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repeated contours in glyphs #463

Open
tamcy opened this issue Sep 21, 2023 · 1 comment
Open

Repeated contours in glyphs #463

tamcy opened this issue Sep 21, 2023 · 1 comment

Comments

@tamcy
Copy link

tamcy commented Sep 21, 2023

In this issue “repeated contours” refers to at least two contours that are completely overlapped. I believe they serve no real purpose and are inadvertently duplicated. After all, static fonts will have these overlapped contours removed during build. While they don’t cause usage problems and are certainly not high priority, it may still be worth fixing them someday so that the problem won’t propagate to new glyphs added in the future due to copy and paste.

image

The following list shows repeated contours as groups. For instance, [1,3] means contours 1 and 3 are repeated (entirely overlapped), and either one of them can be removed.

I first spotted 2 pairs of repeated contours in 肧 (uni80A7-HK), and decided to write a program to find glyphs with similar issue. So the list should be complete.

Glyph Repeated contour groups
uni2600 (The whole glyph)
uni3229-JP [1,3];[2,4]
uni3345 [4,6];[5,7]
uni465B-CN [4,16]
uni5065uE0101-JP [11,12]
uni56A5-TW [1,14]
uni5964-JP [1,12]
uni80A7-HK [1,3];[2,4]
uni98BC-HK [5,19]
uniAEF4 [1,8]
uniB1F5 [1,7]
uniB367 [5,7];[6,8]
uniC254 [3,10];[4,11]
uniC3A4 [1,10];[2,11]
uniCAC5 [3,9];[4,10]
uniCAC6 [3,9];[4,10]
uniCAC7 [3,9];[4,10]
uniCAC8 [3,9];[4,10]
uniCAC9 [3,9];[4,10]
uniCACA [3,9];[4,10]
uniCACB [3,9];[4,10]
uniCACE [3,9];[4,10]
uniCAD5 [3,9];[4,10]
uniCAD7 [3,9];[4,10]
uniCB34 [3,9];[4,10]
uniCB35 [1,7];[2,8]
uniCB36 [3,9];[4,10]
uniCB37 [1,7];[2,8]
uniCB38 [1,7];[2,8]
uniCB39 [1,7];[2,8]
uniCB3A [1,7];[2,8]
uniCB3B [1,7];[2,8]
uniCB3E [1,7];[2,8]
uniCB43 [1,7];[2,8]
uniCB45 [2,8];[3,9]
uniCB47 [1,7];[2,8]
uniCB51 [2,8];[3,9]
uniCB52 [2,8];[3,9]
uniCB53 [1,7];[2,8]
uniCB54 [2,8];[3,9]
uniCB55 [2,8];[3,9]
uniCB56 [2,8];[3,9]
uniCB57 [2,8];[3,9]
uniCB5A [2,8];[3,9]
uniCB5F [2,8];[3,9]
uniCB61 [2,8];[3,9]
uniCB63 [2,8];[3,9]
uniCBC0 [4,10];[5,11]
uniCBC1 [2,8];[3,9]
uniCBC2 [2,8];[3,9]
uniCBC3 [1,7];[2,8]
uniCBC4 [2,8];[3,9]
uniCBC5 [2,8];[3,9]
uniCBC6 [2,8];[3,9]
uniCBCA [2,8];[3,9]
uniCBCF [2,8];[3,9]
uniCBD1 [2,8];[3,9]
uniCBD3 [2,8];[3,9]
uniCBDC [4,10];[5,11]
uniCBDD [2,8];[3,9]
uniCBDE [2,8];[3,9]
uniCBDF [1,7];[2,8]
uniCBE0 [2,8];[3,9]
uniCBE1 [2,8];[3,9]
uniCBE2 [2,8];[3,9]
uniCBE3 [2,8];[3,9]
uniCBE6 [2,8];[3,9]
uniCBEB [2,8];[3,9]
uniCBED [2,8];[3,9]
uniCBEF [2,8];[3,9]
@punchcutter
Copy link
Member

Thanks for the list. I've noticed a few here and there before. I've fixed most of these now. I'll get them all fixed for the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants