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

[serialize] Add object_t, link_t, and snapshot_t

[map] Add another TODO item

[map] Add TODO

[serialize] Add unused 'tail'

[serialize] Add ran_out_of_room

Split sanitize and dispatch into their own files

Move serializer to hb-serialize.hh

Whitespace

[map] Deref pointers before equality check

[map] Templatize hb_map_t

Template name is hb_hashmap_t<K,V>.

Add hb_deref_pointer()

[iter] Fix bug in hb_any() and hb_none()

[iter] Port remaining "for (auto" instances to daggers

[iter] One more dagger

[iter] Remove more wrong &&'s

Sigh...

[iter] More daggers

[iter] More daggers

[iter] One more dagger

Remove HB_DEBUG_WOULD_APPLY

Not that useful.

[iter] Port more to daggers

Whitespace

Simplify code

[iter] Port one more function to dagger

[iter] Port more code to daggers

[iter] Port two more functions to daggers

Fix pair signature

Oh well.  Again, who does fully understand what is right???

[iter] Add hb_unzip()

Whitespace

[iter] Port two more loops to dagger

[iter] Remove unneeded &&

Next commit needs this.  I never fully get this, sigh.

Whitespace

Remove hb_assign()

Not needed anymore.  We just use operator= now.

[C++11] Remove IntType::set() in favor of operator=

[vector] Accept all types in push(...)

Let assignment operator worry about conversion.

[C++11] Use type aliases for template partial instantiations