freetype icon indicating copy to clipboard operation
freetype copied to clipboard

multiple fixes: prevent crash from nil loca, fix Height metric, cache advances, more robust to bad kern issues

Open rcoreilly opened this issue 7 years ago • 6 comments

These are all extensively tested in context of GoGi project: https://github.com/goki/gi and fix a number of important issues to make it work. Let me know if there are any questions on anything! Thanks,

  • Randy

rcoreilly avatar Aug 17 '18 05:08 rcoreilly

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

googlebot avatar Aug 17 '18 05:08 googlebot

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

googlebot avatar Aug 17 '18 05:08 googlebot

I signed it!

rcoreilly avatar Aug 17 '18 05:08 rcoreilly

CLAs look good, thanks!

googlebot avatar Aug 17 '18 05:08 googlebot

CLAs look good, thanks!

googlebot avatar Aug 17 '18 05:08 googlebot

Sorry I forgot to check existing PR's and see that this is partially redundant with #32 -- seems like this package is not being actively maintained. Therefore, I'll plan to keep my fork at https://github.com/goki/freetype up-to-date if anyone wants to contribute PR's there. Without the above fixes (or #32) the font height computation is definitely incorrect, and it is not usable for browsing fonts on Linux because of the crashing. I'll merge #49 and #50 now -- will be handy for SVG!

rcoreilly avatar Aug 17 '18 06:08 rcoreilly