[dfont] Disable null-processsing for offsets

An offset to unsized arrayis not safe to be redirected to our fixed-sized
null pool.  Plus, we want to reject, not repair, bad-looking dfonts.

[unicode] Move Fitzpatrick hack from ot-layout into unicode.hh

Minor

Fix builds

[aat] Add enums for pre-defined state and classes

Not sure how I didn't add before...

Don't dereference offset before check_struct()

[dfont] Also check dataLen range in sanitize

Just to disagree with myself re being done with this code...

[dfont] Solve the mystery +2 offset thing!

Previously, ResourceForkHeader was defined as 30 bytes, having the typeCountM1 as last member.
There was a mysterious offset-by-2 in the code, derived from FontTools and JDK code this was
ported from.

In testing, I observed that typeListZ offset is actually 28.  Suggesting that the typeCountM1
does NOT actually belong to ResourceForkHeader, but belongs to the array itself.  Adjusting for
that resolves the mystery +2 offset hack, so everything is clean and good now.

This, concludes my dfont hacking.  The code looks great now, and I'm happy to leave it.
Fuzzers might disagree though, we will see!

Rename HeadlessArrayOf::len to lenP1

So it doesn't accidentally match our templates, etc.

[dfont] Some more

[dfont] Push methods further down

[dfont] Minor

[dfon]t Sanitize OpenTypeFontFace

[dfont] Minor

[dfont] Make test pass

Offset 0 is not null in this context.

Allow Offset<>'s that have no 0==null

[dfont] Minor

[bytes] Rename content to arrayZ

[dfont] Clean up sanitize()

I don't think I broke anything.  Fuzzers will let me know..

Fix OffsetTo::sanitize() after reshuffling

OffsetTo::sanitize() Add version with three user_data

OffsetTo::sanitize() more shuffling

OffsetTo::sanitize() reshuffling

[dfont] Some renaming, plus add link to reference doc

[dfont] Sanitize only sfnt resources as OpenTypeFontFile

[dfont] Minor

[dfont] Fix test expecatation and minor touch up

I have no way to authoritatively know, but looks like test font only has one
face.  So, adjust test expectation instead.

[dfont] Add test

[dfont] Re-enable and fix offset handling

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

Whitespace

[ft] Invalidate advance cache if font size changed

[ft] Cache advances

I decided to always use the cache, instead of my previous sketch direction
that was to only allocate and use cache if fast advances are not available.
The cache is a mere 1kb, so just use it...

TODO: Invalidate cache on font size change.

Fixes https://github.com/harfbuzz/harfbuzz/issues/651
Fixes https://github.com/harfbuzz/harfbuzz/pull/1082

[util] Add --ft-load-flags

Useful for performance testing.

Not hooked to cairo yet.  Just changes shaping, not rasterization.

[util] Add -n shorthand for --num-iterations

Meh.

[ft] Towards caching slow get_h_advance results

Related to https://github.com/harfbuzz/harfbuzz/pull/1082

[ft] Add advances() callback

[ot-font] Rename

[ot-font] Add advances() callbacks

[cache] Minor

Revert "Remove unused hb_cache_t"

This reverts commit 473b17af4d421f4ce7ac18c769731bb2aa4088f8.

Updates to recent changes.

[random] Switch to 32bit RNG

[random] Shuffle

Make --features rand=1 available to the user

Use rand=255 to mean "randomize".

Part of https://github.com/harfbuzz/harfbuzz/pull/803

Respect user's wish if they set rand feature manually

Except if the set it to 1, which would mean "randomize"... Ugly.

Minor clean up of 'rand' patchset

Don't seed the RNG from the contents of the buffer

Test 'rand'

Allow requesting a specific glyph for 'rand'

Randomization only happens by default. If the user specifies a value for
'rand', that value is respected.

Implement 'rand'

[uniscribe] Fix build

Add HB_FEATURE_GLOBAL_START/END

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

New API:
HB_FEATURE_GLOBAL_START
HB_FEATURE_GLOBAL_END

Add (unused) hb_array_t<>

Minor

Port rest from VAR to UnsizedArrayOf<>

Simplify sanitize->check_array()

Fix a bug in CBDT sanitize, and redundant check in avar.

Port some VAR arrays to UnsizedArrayOf<>

Fix avar sanitize().

[indic] Cache hb_options().uniscribe_bug_compatible on indic_plan

Unbreak HB_OPTIONS

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

[indic] Do NOT allow matra after Halant,ZWJ

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

Devanagari regresses 12 tests, and Gujarati 2.  See:

  https://github.com/harfbuzz/harfbuzz/issues/556#issuecomment-419957472

New numbers:

BENGALI: 353725 out of 354188 tests passed. 463 failed (0.130722%)
DEVANAGARI: 707299 out of 707394 tests passed. 95 failed (0.0134296%)
GUJARATI: 366353 out of 366457 tests passed. 104 failed (0.0283799%)
GURMUKHI: 60729 out of 60747 tests passed. 18 failed (0.0296311%)
KANNADA: 951300 out of 951913 tests passed. 613 failed (0.0643966%)
MALAYALAM: 1048136 out of 1048334 tests passed. 198 failed (0.0188871%)
ORIYA: 42327 out of 42329 tests passed. 2 failed (0.00472489%)
SINHALA: 271596 out of 271847 tests passed. 251 failed (0.0923313%)
TAMIL: 1091754 out of 1091754 tests passed. 0 failed (0%)
TELUGU: 970555 out of 970573 tests passed. 18 failed (0.00185457%)

Fix sanitize or Context Rule

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

Revert "[subset] Disable GSUB/GPOS subsetting for now"

This reverts commit 616fd34a69bb69bc35c7e4ea939e71c3ea2e92cb.

1.9.0

[subset] Disable GSUB/GPOS subsetting for now

So I can get a release out.  I haven't debugged those yet, and they
are producing bad tables.

[atomic] Fix pointer type passed to InterlockedExchangeAdd()

John Emmas reported on mailing list that build with MSVC is failing.
Not sure why bots don't catch it, but this should fix.

Use enum instead of "static const" in class scope

Technically, static const needs an out-of-class definition.  Eg:

  CXXLD    libharfbuzz-subset.la
Undefined symbols for architecture x86_64:
  "OT::FeatureVariationRecord::min_size", referenced from:
      bool OT::GSUBGPOS::subset<OT::PosLookup>(hb_subset_context_t*) constin libharfbuzz_subset_la-hb-subset.o
      bool OT::GSUBGPOS::subset<OT::SubstLookup>(hb_subset_context_t*) constin libharfbuzz_subset_la-hb-subset.o
  "OT::Record<OT::LangSys>::min_size", referenced from:
      OT::Script::subset(hb_subset_context_t*) constin libharfbuzz_subset_la-hb-subset.o
  "OT::IntType<unsigned short, 2u>::min_size", referenced from:
      OT::Script::subset(hb_subset_context_t*) constin libharfbuzz_subset_la-hb-subset.o
      OT::RecordListOf<OT::Feature>::subset(hb_subset_context_t*) const  in libharfbuzz_subset_la-hb-subset.o
ld: symbol(s) not found for architecture x86_64
collect2: ld returned 1 exit status
make[4]: *** [libharfbuzz-subset.la] Error 1
make[3]: *** [all-recursive] Error 1
make[2]: *** [all] Error 2
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Exited with code 2

Try fixing bots

[subset] Fix div-by-zero

[subset] Fix hdmx subsetted size calculation.

Fix older compilers re '>>' at nested template declarations

[subset] Hook up GSUB/GPOS, but still disabled

They are still in the drop list.

[subset] Implement subsetting of SingleSubst

[subset] Keep glyph set in plan

Should remove the vector version at some point...

[subset] Towards subsetting SingleSubstFormat1

Why does subset plan not have a hb_set_t of glyphs?

[subset] Wire up subset() call down to subtables

Templatize Lookup::sanitize()

[subset] More

Templatize GSUBGPOS::sanitize()

[subset] Fix serialize_subset() calls

Ouch.

Add intersects() method to GSUB/GPOS lookups

[subset] Minor

Remove Lookup::subset().

[subset] Towards GSUB/GPOS subsetting

Add subset() call for GSUBGPOS struct and its dependencies.
Not hooked up anywhere.

[serialize] Add reset()

[subset] De-templatize hb_subset_context_t

We're going to (finally) use virtual methods for hb_serialize_context_t
customization, so don't need to carry a Serializer template variable
around...  Simplifies code.

[serialize] Add a couple small methods

[subset] Add hb_subset_context_t<>

[subset] Add hb-subset-input.hh and make hb-subset.hh toplevel include

Minor

Shuffle

[subset] Clean up hb_subset_input_t API

[subset] Bail if subsetting a table failed

[subset] Remove unused hb_subset_profile_t

We might reintroduce it later, but for now remove, as it was unused.
Some things that should have been in this object (drop_hints, etc)
are already in hb_subset_input_t.  So, for now, keep everything there.

[ot-font] Clean up cmap with gid=0

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

Delete pkg.m4

pkg.m4 is provided by pkg-config, which is a requirement to build, and this copy is ancient.  Delete it to avoid it being used instead of the host copy.

Don't declare extern symbols as inline

clang -O3 was completely removing _get_gdef(), causing link
failure when needed from another compilation unit.  Surprisingly,
"extern inline" didn't fix it.

Add explicit to hb_auto_t 1param constructors

Fix distcheck

Explicitly pass reference in hb_auto_t constructor

Fixes clang bots as well as fuzzer issue.

[ot-face] Remove tables_t. The face_data_t is what tables_t was

Free up hb-ot-face.hh from includes

There might be a better way to do this, but I couldn't find...

[face] Use saved cmap in collect_unicodes() APIs