Merge branch 'master' of https://github.com/harfbuzz/harfbuzz

Add HB_NO_SUBSET_LAYOUT

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Revert "Hide symbols in hb-iter"

This reverts commit 98f14c4cdb837a962083a6702f401d41b4c1ec5c.

Same as previous commit.

Revert "Hide more symbols"

This reverts commit 2e86d50915cf1a791da9acb95245aa820a3d70f4.

I think the setup that caused me to do this is faulty and not hiding inlines.

Hide symbols in hb-iter

Painful.  All template methods need to be explicitly hidden :(.

Maybe we should switch to -fvisibility=hidden pragma.

A LOT more to go.

Hide more symbols

Exposed by:

$ make -j5 CPPFLAGS="-O0" CXXFLAGS=-flto=thin LDFLAGS=-lc++ && ./check-symbols.sh

Hide a few more symbols

Exposed by:

$ make CPPFLAGS=-O0

Hide a few static methods

Looks like static methods that do not get inlined end up exported.
We have a lot more.  Need to protect all at some point.  Wish there
was an easier way, like the visibility flag we pass that automatically
hides all inline methods.

Was exposed by check-symbols.sh when compiling on OS X 10.14 with:

$ make CPPFLAGS=-Oz CXXFLAGS=-flto=thin LDFLAGS=-lc++

Add HB_NO_OPTIONS

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_NAME_TABLE_AAT

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_FONT_BITMAP

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_FONT_CFF

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_SHAPE_COMPLEX_HEBREW_FALLBACK

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_SHAPE_FALLBACK

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_SHAPE_COMPLEX_THAI_FALLBACK

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_LAYOUT_BLACKLIST

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Add HB_NO_OT_SHAPE_COMPLEX_VOWEL_CONSTRAINTS

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Update gtk-doc annotations for inout counts on various getter functions.

[Docs] Minor edits to gtk-doc inline comment review.

[Docs] Minor; fix formatting for gtk-doc multiple-annotations.

[Docs] Annotate gtk-doc formatting with some un-annotated (out)s.

[Docs] Fix gtk-doc formatting for (out) and (inout).

[Docs] Fix gtk-doc references to 'kern' table functions, clarifying that GPOS is not examined.

[Docs] Add inline gtk-doc documentation of GDEF glyph classes.

[Docs] Add inlind gtk-doc comments for hb-ot-layout functions.

Use injected class name to simplify macros

[C++11] Use deleted methods

Rename a few macros

[arabic] Disable fallback shaping if HB_NO_OT_SHAPE_COMPLEX_ARABIC_FALLBACK defined

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Don't install ot-font funcs on new fonts if HB_NO_OT_FONT defined

Currently linker cannot GC hb-ot-font completely because we install
it on fonts by default.  Don't do that if HB_NO_OT_FONT defined.

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

Don't use any default unicode funcs if HB_NO_UNICODE_FUNCS is defined

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

[coretext] Fix unused-variable error

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

Fix MSVC C4068 warning (#1656)

replace test font SourceHanSans with its subet

Fix MSVC C4138 warning (#1657)

Minor

[GDEF] Don't assume glyphlist is sorted

As was hit by the fuzzer.

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

[algs] Add hb_min() and hb_max()

[serialize] Make putting breakpoint on out-of-memory easier

[iter] Tweak SFINAE again

Don't think we need hb_is_same().

[array] Add compy assignment operator since copy constructor is explicit

[array] Add default copy constructor

MSVC seems to need it.

[iter] Use different SFINAE scheme to make MSVC happy

From Orvid King: TLDR; MSVC has some issues using sizeof(declval<T>()) for
SFINAE of templated types, so I just used SFINAE in a different context where
MSVC doesn't have the issue.

[meta] Add hb_is_same()

[serialize] Actually reclaim storage from duplicate objects

[serialize] Port to use object pool

Tested, but feels fragile :(.

[pool] Uses memset() instead of assigning Null()

Assignment is invalid on invalid object.

Add hb_pool_t<> for pooled memory allocation

[serialize] Disable packed_map again

Ugh.  Need to think of something else.

[map] Protect more against pointer deref

More

[serialize] Fix hb_hashmap_t<> for pointers and use in packed_map

[iter] Remove passing pointer to hb_iter()

While doable with hb_deref_pointer() as well, we also would then
need to do it in a ton of places.  Not worth it / messy.

[array] Use dagger for hashing array

Also switch to better mixing.

[iter] Accept pointers in hb_iter()

No idea how to avoid dupicating code.  Was hoping hb_deref_pointer()
would do it, but looks like a pointer can't bind to a universal
reference T&&.  Humm.

[serialize] Towards maintaining packed_map

[serialize] Minor

[serialize] Handle non-nullable offsets

[serialize] Movce empty-object handling earlier

[serialize] Minor

[serialize] Allow offset links that have base offset from the object base

Rarely used, but used, in name table or similar constructs.

[test][iter] minor, fix double promotion warning

[test] minor, c style comments

[iter][test] Add another test for hb_reduce

Different initial and accumulator types

[serialize] Switch to tetris-packing

[serialize] Fix linking

[serialize] Minor

[cmap] Minor

[subset] Call serialize start/end around it

To be cleaned up.

[serialize] Assert stack

[serialize] Add add_link() to add link

[serialize] Add default template type to push()

[iter] hb_reduce, accumulator with a different type

[iter] Implement hb_reduce

[serialize] Implement linking

Untested!

Move BEInt to hb.hh

I knows...

[iter] Fix up build, ouch

Yeah, some things not very clear...

Don't skip setting the .end field of the first range

Fixes a bug in CoverageFormat2::serialize whereby the first range
was not serialized correctly if it consists of only a single glyph ID.
This broke shaping of U+0626 in the Arabic fallback shaper, because it
is not found in the coverage table of the 'init' and 'medi' lookups.

Also fix similar bug in ClassDefFormat2::serialize, noted during code
inspection (I haven't observed a case that was actually affected by
this, but it looks broken).

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

[iter] Adjust hb_copy() and use it

Untested.

Remove coretext_aat shaper (#1581)

coretext_aat was a temporary shaper to redirect shaping of AAT fonts
to CoreText and leaving the rest for HarfBuzz.  As HarfBuzz now supports
AAT and Chrome now actually ships that on a stable version on macOS,
we no longer care about such use-case.  If a client really wants 100%
metrics compatibility with CoreText better to use it directly or through
our API.  Replicating the same behavior still is possible using
hb_shape_full, something we don't care or like to offer anymore.

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

[serialize] Start implementing linking

[serializer] Implement dedup!

Implement hashing of objects

Should be improved for hb_bytes_t.

Fix hb_hash(pointer)

[serialize] Don't insert empty object into tree

[serialize] Towards maintaining hashmap

[serialize] Only pack main object if there are other objects

Avoids a memmove for tables that don't use the object packing mechanism.

[serialize] Copy both sides of the buffer

[serialize] Simplify copy

[serializer] Unbreak for now

[map] Shuffle fini code

[vector] Add move semantics

[vector] Add some move and forwarding

[meta] Add hb_move and hb_forward ala std::

[serialize] Flesh out packing

[vector] Add copy constructor and assignment operator

[serialize] Minor

[serialize] Add packed_map

[serialize] Start fleshing out object stack

[serialize] Simplify propagate_error()