Add 10.13.6 aat fonts tests and bot (#1409)

[aat] Add test for recent regression

[tests] Redo test runner logging a  bit

[tests] Minor

[tests/shaping] Allow comments in test files

Line should start with "# ".

[tests] Rename

[morx/kerx] Fix sanitize regression

Broke in 8dcc1913a1670ede7b124f7b5b775d7ab8791386

If sanitizer is left with another object, it wouldn't work.

Better fix coming soon.

[test] Enable more of aots tests (#1408)

The ones commented out in this change should've be passed :/
a closer look is needed.

Add Adobe AOTS tests (#1395)

Annotated OpenType Specification or aots, https://github.com/adobe-type-tools/aots
provides a set of tests for OpenType specification, this change add those tests in addition
to modified version of their HarfBuzz test runner for generating harfbuzz project specific tests.

[color] Use SortedUnsizedArrayOf<>

[trak] Coment

[arrays] Add not_found to reference bsearch as well

[arrays] Improve bfind() interface

Much more useful now. :)

[fuzzing] Add tests for previous commit

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

[kerx/morx] Make sure object length is sanitized before accessing it

[arrays] Port ArrayOf.qsort() and hb_vector_t.qsort() to hb_array_t

[arrays] Port ArrayOf.qsort() to hb_array_t's

[arrays] Add copy-constructor to hb_array_t and hb_sorted_array_t

[arrays] Port hb_vector_t.qsort() to hb_array_t's

[arrays] Remove one flavor of hb_vector_t.qsort()

[arrays] Add (unused) SortedUnsizedArrayOf<>

[arrays] Port SortedArrayOf.bsearch/bfind to hb_sorted_array_t's

[arrays] Port hb_vector_t.bsearch/bfind to (new) hb_sorted_array_t's

[arrays] Port (unused) ArrayOf.lsearch() to hb_array_t's

[arrays] Port hb_vector_t.lsearch() to hb_array_t's

[arrays] More

[arrays] Update ArrayOf.lsearch()

Currently unused apparently

[arrays] Change argument type of cmp called by hb_vector_t.bsearch()

Towards consolidating all array bsearch/...

Usermanual: clusters chapter; add brief grapheme definition and clarify monotonous cluster handling.

Usermanual: clusters chapter, minor updates.

Usermanual: expand clusters chapter.

Change ArrayOf.bsearch() return semantics

Towards consolidating all array bsearch/...

Minor

Whitespace

Assert that item-type of arrays have static size

[vector] Rename

[aat] Minor

Add hb_static_size(T)

Add template-function convenience macros

Add null bytes for CmapSubtableLongGroup

Remove lsearch for small TableDirectorys

[arrays] Add as_array() to hb_vector_t<>

[arrays] Add as_array() to ArrayOf<>

[kern] Don't enforce length of last subtable

Merge pull request #1407 from harfbuzz/at-sign

Don't canonicalize '@' to '-' in language tags

Don't canonicalize '@' to '-' in language tags

Fixes #1406.

[aat] Update expectation

[aat] Disable mark advance zeroing if kern table has state-machines

Geeza Pro for example, relies on that for fancy mark positioning.

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

Minor

[aat] Add macOS specific tests (#1404)

[use] Minor clarification

Minor tweak to FLAG64

Merge pull request #1291 from harfbuzz/use-reordering

[use] Fix reordering

[cmap] Move code around

[morx/kerx] Limit range to subtable when sanitizing

[sanitizer] Add reset_object(), make set_object() do bounds-check

Affects morx/kerx run-time only currently.  Will adjust their sanitize next.

[kerx] Make sure subtables are non-zero-length

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

[GSUB] Don't flush glyphset during recursion in closure()

See comment.

Supercedes https://github.com/harfbuzz/harfbuzz/pull/1401
Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11366

[subset] add fuzzer testcase.

[set] Rename

Enforce requiring null_size even if min_size is 0

This concludes null-size enforcement changes

Allow non-nullable OffsetTo<> to non-Null'able objects

[glyf] Don't mark structs UNBOUNDED

See comments.

Mark UnsizedArrayOf<> as UNBOUNDED

Since min_size is 0, Null() still accepts this type.

[ot-shape] Simplify logic

Move

[aat] Zero mark advances if decided so even if there's cross-kerning

Cross-kerning can only take care of positioning vertically.  It doesn't
adjust mark advance...

[aat] Tweak fallback positioning logic when applying morx

Such that for Indic-like scripts (eg. Khmer), we don't do any fallback mark
advance-zeroing / positioning, but we do for Latin, etc.  Reuses preferences
of our script-specific OpenType shapers for those.

Fixes regression: https://github.com/harfbuzz/harfbuzz/issues/1393
Which means, fixes again: https://github.com/harfbuzz/harfbuzz/issues/1264
While not regressing: https://github.com/harfbuzz/harfbuzz/issues/1357

[myanmar] Minor move

[myanmar] If there's no GSUB table, pick myanmar shaper

Needed for morx+kern mark-zeroing interaction.  All other scripts
work this way.

[myanmar] Remove myanmar_old shaper

Over time it has become the same as default shaper.  So, remove.

Fix spurious gcc warnings

../../src/hb-null.hh:53:39: warning: enum constant in boolean context [-Wint-in-bool-context]

More

This makes more of the gcc spurious warning:

../../src/hb-null.hh:53:39: warning: enum constant in boolean context [-Wint-in-bool-context]

But not going to let that defeat correct code.  Type to switch to clang
as my main compiler...

Fix up recent change

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

Minor

Minor

Disallow taking Null() of unbounded structs

Not sure I've marked all such structs.  To be done as we discover.

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

Use Type::null_size for our structs in Null(), sizeof() for other types

One more time..

Fix bots happy again, hopefully

So, our fallback static_assert cannot be had more than once per line
of source.

Revert alignof() == 1 check

Bots not happy with using "this" inside assertion...

This reverts 2656644887e77a9d814bb12374af3c26b42fd935

..

Minor

Minor

Minor

Remove DEFINE_SIZE_ARRAY2

Check alignof() structs are 1

Whitespace

Merge pull request #1400 from harfbuzz/sharada-sandhi-mark

Test U+111C9 SHARADA SANDHI MARK

Test U+111C9 SHARADA SANDHI MARK

Merge pull request #1399 from harfbuzz/sharada-sandhi-mark

Fix USE categories for U+111C9 SHARADA SANDHI MARK

Allow defining HB_USE_ATEXIT to 0

That's better use of that value than requiring extra macro HB_NO_ATEXIT

Fix USE categories for U+111C9 SHARADA SANDHI MARK

[var] Deprecated axis enumeration API and add new version

New version has axis flags.

New API:
+hb_ot_var_axis_info_t
+hb_ot_var_find_axis_info()
+hb_ot_var_get_axis_infos()

Deprecated API:
-HB_OT_VAR_NO_AXIS_INDEX
-hb_ot_var_axis_t
-hb_ot_var_find_axis()
-hb_ot_var_get_axes()

Remove newly-added -hb_ot_var_axis_get_flags()

[var] Make sure hb_ot_var_axis_flags_t is int-sized

[var] Move code

Merge pull request #1397 from harfbuzz/small-emoji-table

Shrink the emoji table by merging adjacent ranges

Shrink the emoji table by merging adjacent ranges

[var] Fix type of coords returned

Ouch.  Wonder how none of the bots caught the float->int truncation.