Merge branch 'master' into cff-subset

fixed Charset format selection

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

ensure fdmap initialized as identity for single-FD (non-CID) fonts

Merge branch 'master' into cff-subset

drop dotsection as hint along with test case

[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

use 2-byte offset instead of 4-byte for local subrs

more than enough since local subrs immediately follow its corresponding private dict, as the result 2-byte redunction for each font dict with local subrs
updated api test expected subset fonts accordingly

fixed a subset bug when no font dicts reduce

[subset] minor, adjust spaces

[math] minor, adjust spaces

reject nested seac

Merge branch 'master' into cff-subset

fixed CFF1 subset of std encoding & non-std charset combo

[base] Add TODO items

[shaper] Rename

[ot-shape] Move code around

Merge branch 'master' into cff-subset

Implemented seac for extents & subset along with API tests

[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

Merge branch 'master' into cff-subset

changed Adobe company name

[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.