caryll / otfcc

Optimized OpenType builder and inspector.
Apache License 2.0
422 stars 63 forks source link

Round-trip Dump and Build appears to change GPOS kerning #58

Open MalcolmWooden opened 5 years ago

MalcolmWooden commented 5 years ago
original

Original font has no kern here.

otfccbuild

After Dump and Build a new kerning pair has appeared.

be5invis commented 5 years ago

Json difference?

be5invis commented 5 years ago

Also OTFCC does not support legacy kern table. Only GPOS would be processed.

MalcolmWooden commented 5 years ago
kopik-bold

First lookups of GPOS kerning from original.

kopik-bold_otfcc096

First lookups of GPOS kerning after Dump and Build.

be5invis commented 5 years ago
    Looks like a bug when generating individual-based lookup from class-based...

On Fri, Dec 28, 2018 at 10:27 PM +0800, "MalcolmWooden" notifications@github.com wrote:

First lookups of GPOS kerning from original.

First lookups of GPOS kerning after Dump and Build.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

be5invis commented 5 years ago

@MalcolmWooden Could you please provide me the file so I can test?

MalcolmWooden commented 5 years ago

Here is the font that I used. Kopik-Bold.otf.zip

be5invis commented 5 years ago

image

I tried one roundtrip (using Master version) and does not have any problem. I know there are some older version that breaks kerning lookups. Maybe you can try to build with Master.

MalcolmWooden commented 5 years ago

Sorry but I am not familar with building projects with Premake and Lua. Do you have a Master Mac O/S version you can upload?

be5invis commented 5 years ago

@clerkma

clerkma commented 5 years ago

otfcc-macos.x64-0.10.3.zip