Added CFF1 J font subset to api test

Also fixed FDSelect3_4::get_fd
Fixed test_subset_cff1_strip_hints (wrong font names!)

fixed memory leaks

in CFF1 & CFF2 accelerators, VORG subsetter

Added CFF1 & CFF2 api tests

Merge branch 'master' into cff-subset

[circleci] Disable msan and ubsan builds

https://github.com/harfbuzz/harfbuzz/issues/1174
https://github.com/harfbuzz/harfbuzz/issues/1175

Ignore weak symbols in check-symbols.sh

Some clang versions leave a std::round(float) weak symbol around...

[docs] A few improvements

If we wrote just this much every day...

Readjust Hebrew composition again

Revert change that would decompose text if GPOS mark feature is available

https://github.com/harfbuzz/harfbuzz/issues/653#issuecomment-423905920

Fix normalization

https://github.com/harfbuzz/harfbuzz/commit/62d1e0852a5549a1b510ad46a4b89f12730bb708#commitcomment-30613091

Simplify ZWJ-skipping a bit

Towards disabling ZWJ-skipping in certain GPOS contexts.

Part of https://github.com/flutter/flutter/issues/16886

Reinit skippy iters when auto_zwj / auto_zwnj change

Ouch.  How did we not hit this bug before...

Prefer decomposed form if font has GPOS mark feature

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

[ot-font] Fix leak

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

I'm glad we have leak-detector bots now.

Add bots with address- and thread-sanitizer

src/hb-blob.cc: Fix mmap functionality with UWP.

[test] Fix printing subprocess error messages

Decode the string as Python 3 returns bytes string, and also don’t
assume that it ends with a new line.

Fix test-subset-* leak issues

See also https://github.com/harfbuzz/harfbuzz/pull/1169

Fix test-ot-math leak issue

See also https://github.com/harfbuzz/harfbuzz/pull/1169

Fix test-set leak issue

See also https://github.com/harfbuzz/harfbuzz/pull/1169

Fix test-font leak issue

See also https://github.com/harfbuzz/harfbuzz/pull/1169

fixed subroutinized CFF2 flattening

and some code cleanup

Added desubroutinize option to generate-expected-outputs.py

Currently hb-subset always desubroutinizes CFF fonts.
Note that even with this option fonttools won't generate the expected subset CFF fonts. They were manually created & tweaked to match hb-subset output.

Added VORG struct and its subsetter

new file: hb-ot-vorg-table.hh

[morx] Fix mark_set check

Added SourceHanSans-Regular for CFF subset test

Disabled FDSelect serialization to format 0 for compatibility with fonttools

[aat] Minor

[coretext] Minor

[morx] Implement forward/backward processing

We reverse too many times. Can be optimized. But I doubt many fonts
use reverse lookups, so doesn't matter.

Other than not applying user features, this completes morx table
implementation.

[morx] Implement horiz-only/vert-only subtables

[morx] Improve buffer position in Insertion w DontAdvance and wo

Just speculation. Needs tests to determine which is correct behavior.

fixed char widths & standard Encoding in subset CFF

also merged flush_n_args to flush_args into one
takes start index of the arguments on stack instead of count

Added SourceSansPro-Regular along as CFF full-font test case

derived "expected" subset fonts from fonttools then manually tweaked further so they resemble hb-subset output

include curve control points in extents

this makes results consistent with FT
also optimized handling of path end points

fix static_assert failure in OffsetTo<TopDict>

Merge branch 'master' into cff-subset

fix build attempt ^3

fix attempt ^2: unuse INT32_MAX/MIN

fix build attempt

[morx] Implement InsertionChain

implemented get_glyph_extents with CFF1

Implemented path operators in a new struct PathProc hooked up to CSOpSet
Added current point to CSInterpEnv

Don't enforce a native direction for Old Hungarian

Fix installation of cmake config when building with cmake (#1161)

When building with the auto tools, the manually mantained
harfbuzz-config.cmake is installed. When building with cmake, we can use
cmake to generate the correct config files for us and install them.

Revert forcing use of single-parameter static_assert()

Some clang versions define static_assert as a macro apparently, so we cannot
redefine it...

This reverts commit 94bfea0ce6a7b4d5641c198d50751748a353df11.
This reverts commit 4e62627831e7457ed60ff87712570065b14b200a.

One more bot fix

Fix build, another try

Fix bots

"Unused private member" warning turned error.  ugh.

Enforce single-param static_assert() only

So we don't accidentally break it again.

Remove unused try

Fix static_assert

Disallow null-enabled offsets to unsized structures...

...like UnsizedArrayOf<>.

This fixes a class of crasher bugs, mostly with color and AAT tables.  We
cannot use nullable offsets to varsized data that does not declare min_size,
because it's nost safe to use our fixed-size null pool for types that have
their size external.  So, use non_null'able offsets for these.

A further enhancement would be to make use of min_size in Null<> itself.
Will try that after.

[morx] Respect default feature settings

Does NOT apply user-selected features.  But at least now enables
correct subtables.

[aat] Change version field

[morx] Add stub for InsertionChain

[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()

Merge branch 'master' into cff-subset

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

add head table to CFF/CFF2

[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

fixed hb_plan_subset_cff_fdselect

FDSelect3/4 sentinel size wasn't taken into account

fixed serialize_fdselect_3_4

fixed bugs in Charset serialization

also some code cleanup

fixed name mixup in Remap::reset

it was causing inconsistent Strings Index subset results

misc bug fixes

Added OpCode_FontMatrix to TopDictOpSet (yet to parse values)
fixed a wrong assert in encode_int

fixed off-by-one error with offSize

fixed a bug

make parse_bcd locale free

Implement parse_bcd

Merge branch 'master' into cff-subset

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