[ci] Build glib and freetype and enable msan bot (#1198)

Minor on test-multithread, align the actual and expected results

[circleci] Add -Wno-reserved-id-macro to clang-everything

[test-multithread] Disable FreeType funcs

Fix fallback kerning to check for current glyph's mask

[circleci] Improve clang-everything bot log, more to come

#1196

[ci] Another try on mingw bot

[ci] Another try on fixing mingw bots

Per https://github.com/Alexpux/MSYS2-packages/issues/163#issuecomment-73555971

[tests] Don't try to get glyph names on failures

As it may cause a race unrelated to the issue actually happened

[ci] Trying to fix mingw bot

As https://github.com/Alexpux/MSYS2-packages/issues/702

Fix pthread fail for real

Using a hack...

Fix bot fails

[morx] Remove mark_set from Insertion

text-rendering-tests test MORX-32 shows that for Insertion, an unset mark is treated
as mark set at 0.  This is unlike the Reordering lookup where un unset mark performs
nothing.

Fixes MORX-32.

[morx] unsafe-to-break in Insertion

Makes MORX-29, MORX-30, MORX-31 pass.

[test/text-rendering-tests] Update from upstream

Fix ubsan bot

More atomic tuneup

[test-multithread] Install ot funcs before filling ref buffer

[test-multithread] Take num-threads and num-iters from command-line

Minor

test-multithread, check the results on every iteration (#1194)

Make test-multithread pass the tsan bot test (#1193)

More atomic fixup

Fixup atomics from recent change

[cache] Use atomic writes in clear()

To help TSan.

Fix infinite loop when walking up the directory hierarchy (#1183)

A single find_package(harfbuzz) line in user code resulted in this loop
getting stuck when _harfbuzz_libdir_iter became "/".

Use clang for compiling freetype for tsan bot

[arabic] Do the joiner-flipping only for rlig feature

See comment.

[ft] Make TSan happy

Add more atomic intrinsics

Rename test

[ci] Compile freetype on tsan and put sanitizer flags on correct places (#1188)

Test freetype funcs on test-multithread (#1187)

Minor, remove the no longer needed comment on test-multithread

Make TSan happy with make_immutable()

Fix test-multithread and increase num_threads to 30

What were you thinking? ;)

Add a multithreaded test (#1184)

[hb-view] Change subpixel bits from 8 to 6

To match FreeType units, such that FreeType gets correct size from us.
This matters more now that we allow setting --ft-load-flags.

Fix iOS build

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

[circle] Add an obsessive clang bot (#1178)

Disable msan bot again

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

Minor

More iter inits

Minor

Remove F_COMBINE

Now I wonder if any bots will be unhappy we calling | in static const initializations...
Or would that cost runtime init?  Our tests don't detect any..

Some more

Minor refactoring

[circleci] Update sanitizer bots with newer clang and Ubuntu version (#1176)

[msan] Enable again to get a build log, and try to play with suppressions

[ubsan] Re-enable now that it passes locally

[check-static-inits.sh] Allow some if ubsan is in effect

[check-symbols.sh] Allow weak objects "V"

ubsan generates these.

Link API test programs with C++ linker

Needed to make ubsan work.

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

[msan] Add MSAN_OPTIONS=exitcode=42

Default exit value is 77, which causes autotools to "skip" test.

[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

[morx] Fix mark_set check

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

[morx] Implement InsertionChain

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

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