fix glyph rendering, handle multiple cmap

vidstige / NRasterizer

OpenType parser in pure c#

Apache License 2.0

38 stars 11 forks source link

fix glyph rendering, handle multiple cmap #51

Open prepare opened 7 years ago

prepare commented 7 years ago

The old version has bugs. It can't render 'Century.ttf' font. so I fixed it. Descriptions are in each node 01-05

test_century

pic 1: '2' glyph, Century font, Gdi+ version

test_century

pic 2: Century font, 240 pt, Notepad [1] vs Typography [2], see red rectangle, this is a characteristic of this glyph (starts with OFF-CURVE point and ends with OFF-CURVE point)

@vidstige, It would be great if you upgrade the project to VS2017 :)

prepare commented 7 years ago

@vidstige, travis-ci => pass :tada: appveyor => failed!, I don't know how to fix that error.

vidstige commented 7 years ago

Thanks! Don't know what's up with the appveyor build... :thinking: Thanks so much for fixing this bug. Will look through code when I get time over.

vidstige commented 7 years ago

Master is also failing, so don't worry too much over that. @tocsoft set this up initially. At least the travis build is passing 👌

prepare commented 7 years ago

@vidstige , Do you have a plan to upgrade the project to VS2017 ?

vidstige commented 7 years ago

@prepare no, I don't. Don't even have VS2017 😬 If you have the changes, feel free to commit. Otherwise I'll might do it whenever I get some time over, if you keep reminding me. 😊