Revert "Add harfbuzzjs build configuration (#1636)" (#1675)

This reverts commit 694cb1beeefe1c54b2e613d2d566a21e248a2c9c.

Add harfbuzzjs build configuration (#1636)

Minor warning fix again

add spaces

fix oss-fuzz issue 14345

silence MVC warnings 3rd attempt

silence MVC warnings 2nd attempt

silence MVC warnings

Minor

[array] Simplify copy assignment/constructor

To fix bogus MSVC warnings:

  c:\projects\harfbuzz\src\hb-array.hh(189): warning C4521: 'hb_array_t<Type>': multiple copy constructors specified [C:\projects\harfbuzz\build\harfbuzz.vcxproj]
  c:\projects\harfbuzz\src\hb-array.hh(189): warning C4522: 'hb_array_t<Type>': multiple assignment operators specified [C:\projects\harfbuzz\build\harfbuzz.vcxproj]

[serialize] Fix offset calculation

[subset] Assert offsets are zero during relocation

If they're not, it's a bug in our subsetting logic somewhere.  So check.

[serializer] Add copy()

Calls obj.copy() or obj.operator=() in that order.

Add HB_RETURN

Comment out ot_languages where fallback suffices

Fix "hb_script_" doc typo

One more auto return type

Rename HB_AUTO_RETURN_EXPR to HB_AUTO_RETURN

Use HB_AUTO_RETURN_EXPR in hb_min/max

Use auto return type for hb_first/hb_second

Use hb_deref_pointer() to reduce number of overloads

Properly prioritize hb_hash()

Move around

Fix priorities

Use type aliasing for meta-functions, ie. those returning a type

Reduce LangTag from 3 language system tags to 1

Reduce NullPool size

[meta] Flesh out hb_invoke()

Comment

[meta] Add hb_invoke()

[meta] Add HB_AUTO_RETURN_EXPR, HB_VOID_RETURN_EXPR, hb_priority, hb_has(), hb_get()

The first three based on range-v3.

Add hb_void_tt<> ala std::void_t

Fix copyright

[fuzzing] Run valgrind with --leak-check=full

[fuzzing] Fail if valgrind is requested but not found

More tweaks to previous commit

Delete assignment operator of OffsetTo<> instead of Offset<>.

In simple ArrayOf<>::sanitize() assert that Type has assignment operator.
Ideally we should SFINAE this and fallback to calling Type::sanitize()
if assignment operator is not available.  But we don't have a case of
that in the codebase.

Delete default assignment operator Offset<>

Add HB_FUNCOBJ()

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

[Usermanual]: fix Tamil error in Why-do-I-need-a-shaping-engine section.

Move static const to post-struct for a function object

Just sending this to bots to see if all happy, then turn it into macro and
apply everywhere.

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

Ugh.  Another try, to unbreak gcc this time!

Jenga.

Move location of HB_UNUSED to make MSVC happy

Fix gcc warning

[ci] Fix macOS glib issue

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