adobe-fonts / source-han-sans

Source Han Sans | 思源黑体 | 思源黑體 | 思源黑體 香港 | 源ノ角ゴシック | 본고딕
Other
14.36k stars 1.3k forks source link

Repeated contours in glyphs #463

Open tamcy opened 1 year ago

tamcy commented 1 year ago

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 commented 1 year ago

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.