[travis] Update Coverity token

Add codecov.io badge

Fix Codacy "issues"

[travis] Another push for codecov.io after setting token

[travis] Add codecov.io

[fvar] Add named-instance API

Fixes https://github.com/harfbuzz/harfbuzz/issues/1241

Fix build

[fvar] Rewrite sanitize

[fvar] Use hb_array_t for axes

[fvar] Minor

[ot-var] Add hb_ot_var_axis_get_flags()

Part of https://github.com/harfbuzz/harfbuzz/issues/1241

New API:
+hb_ot_var_axis_flags_t
+hb_ot_var_axis_get_flags

[ot-var] Add flags

Unfortunate that we don't have room in hb_ot_var_axis_t to expose flags :(.

[ot-var] Use hb_ot_name_id_t

2.1.3

Fix recent commits

Revert b4c61130324455bfd42095b01fa14ac901e441f1

Was causing more trouble than it solved.  We use unsigned for indexing,
and it's not helpful to allow that wrapping to negative integers on
32bit machines.  The only way we could work around it would have been
by accepting int64_t arg, but that's overkill.

Ignore the MSVC 2008 build issue.  We don't support that compiler.

Detect over/under-flow in UnsizedArray::operator[]

Was causing bad substitutions in mort table because of WordOffsetToIndex()
producing negative numbers that were cast to unsigned int and returned as
large numbers (which was desirable, so they would be rejected), but then
they were cast to int when passed to this operator and acting as small
negative integers, which was bad...

Detect overflow.  Ouch, however, now I see this still fails on 32-bit.
Guess I'm going to revert an earlier change.

[aat] Fix mort shaping

Ouch!

2.1.2

Fix vertical fallback space sign

Ouch!

Follow-up to cf203af8a33591c163b63dbdf0fd341bc4606190

Fixes https://github.com/harfbuzz/harfbuzz/issues/1343

Don't apply GPOS if applying morx

That's what Apple does, and it wouldn't degrade our OpenType performance.

Part of https://github.com/harfbuzz/harfbuzz/issues/1348

Prefer morx table if GSUB is empty (no scripts)

Fixes https://github.com/harfbuzz/harfbuzz/issues/1348

Minor

[coretext] Hopefully the last one

[coretext] Another round

[coretext] One more try..

[coretext] One more try

[coretext] Another build fix attemt

[directwrite] Fix build

[coretext] Try to fix

[coretext] Unbreak build

[shaper] Rewrite shaper data code to be more template-driven than macro-driven

[hdmx] Fix bounds checking

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11351

[hdmx] Minor

[subset] Remove invalid glyphs from glypset to retain

Fixes https://github.com/harfbuzz/harfbuzz/pull/1388

[subset] minor, adjust spaces

[math] minor, adjust spaces

[base] Add TODO items

[shaper] Rename

[ot-shape] Move code around

[STAT] Add table parsing (#1384)

[hmtx] Fix signedness issue

Fixes https://github.com/harfbuzz/harfbuzz/issues/1248#issuecomment-438689499

[ot-color/svg] Note that it can be gzipped

Merge pull request #1383 from punchcutter/master

Add test for https://github.com/harfbuzz/harfbuzz/issues/1379

Add Grantha test

[aat] Disable fallback mark advance zeroing and positioning if morx applied

Fixes https://github.com/harfbuzz/harfbuzz/issues/1357

Don't retry creating again and again in lazy_loader if create failed

Still does that if get_null() returns nullptr.  Our shaper data objects
are like that.  Shrug.

Merge pull request #1382 from punchcutter/master

Change USE Category for Grantha Virama

Merge pull request #1380 from kbrow1i/cygwin

Don't use Win32 API on Cygwin

Change USE Category for Grantha Virama
https://github.com/harfbuzz/harfbuzz/issues/1379

Fix Windows build

Use atomic ints for upem and num_glyphs on face

Make atomic types' internal values non-mutable

This resulted from confusion previously...

[ot-face] Add 'head' table

Don't use Win32 API on Cygwin

Cygwin is a Posix platform to the extent possible.  It should use the
Posix API except in special circumstances.

[shape-plan] Remove unused code

[shape-plan] Implement fine-grained caching plans with user-features

Only tag, value, and global-ness of features are considered, not their
start/end offsets.

[shape-plan] Cache shape plans with variations based on variation indices

[shape-plan] Refactor more

[shape-plan] Only use shape-plan key to initialize hb_ot_shape_plan_t

Such that we don't accidentally use info not in the cache key.

[shape-plan] Minor

[shape-plan] Refactor some more

[shape-plan] Refactor more

[shape-plan] Turn hb_shape_plan_proposal_t into hb_shape_plan_key_t

And include it in hb_shape_plan_t itself.

Add hb_memcmp()

[shape-plan] Minor

[shaper] Rename

[fuzzing] Add new test

Rename check_array2() to check_array()

Rename check_array(array, a, b) to check_range()

[kerx] Fix integer overflow in multiply

Fixes https://oss-fuzz.com/v2/testcase-detail/5754863779053568

[kerx] Minor tweak on previous commit

[fuzzing] Remove limited-edition build of libraries

Use normal, production, shared libraries.

Fixes https://github.com/harfbuzz/harfbuzz/issues/1237

[kerx] Protect against stack underflow

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11367

[shape-plan] Minor

[shape-plan] Minor rename

Add variation coords to shape_plan proposal

This is the root cause of bug worked around in 19e77e01bc13f44138e1d50533327d314dd0a018.

Still no shape plan caching for variations though.

Minor

Embed hb_ot_shape_plan_t into hb_shape_plan_t

No other shaper will need shape_plan_data, by definition.  So, remove
abstraction layer and always create hb_ot_shape_plan_t as part of
hb_shape_plan_t.

Remove wrong comment

Finish off eecccc919ce15f60ae7ef9da5cfb311b4aa05c63

Minor

Minor

Revert "Declare Null() constexpr"

This reverts commit 442a72d95ab1fb3a47b486d8d1eb68e909d0ffb8.

Doesn't make sense.  No idea how my local compilers where happy with it!

Minor

Declare Null() constexpr

Don't store to null object

Ouch :).

Remove unused function

[hdmx] Renames

Add hb_blob_ptr_t.destroy()

[hmtx/port] Use hb_blob_ptr_t

[cmap] Port to hb_blob_ptr_t

Although didn't need it...

[color] Port to hb_blob_ptr_t

Fix hb_blob_ptr_t::get_length () as well.

Comment

[shape-plan] Remove use of custom null object

[shape-plan] Make null object all zeros

To remove custom null object next..

[blob] Use default null object

[blob] Change null object memory mode to DUPLICATE

We never rely on that being equal to readonly.  Just not being
writable.  Maybe not even that given that the object is inert.

In prep for next commit, using default null pool.

Add hb_blob_ptr_t

Use in a couple of places.  Push to bots to see how many unhappy before
I convert the rest.

Minor