[circleci] Add an iOS bot (#1233)

[kerx] Comment

[aat] Rename

[morx] Use subtable range for embedded sanitizer here as well

[aat] Fixup recent commit

For 329f2401082011007d9ce12b15ce0225cd267c57

max_ops is signed.

[kerx] Use subtable range for runtime checks

[aat] Set embedded sanitizer max ops really high

Since we consume it legitimately during shaping.

[kerx] Comment

[kerx] Remove junk

[kern/kerx] Adjust bounds check

[kerx] Implement Format2 apply()

Still, not hooked.

Fix check

[kern] Use kern subtable length for sanitizing in the accelerator

[kern] Minor

[kerx] Implement Format0 apply()

Not hooked up to be called yet.

Minor

[kern] Sanitize 4 bytes, not 2

[ft] Use mutex to lock access to FT_Face

Makes our FT-backed hb_font_t safe to use from multiple threads.  Still,
the underlying FT_Face should NOT be used from other threads by client
or other libraries.

Maybe I add a lock()/unlock() public API ala PangoFT2 and cairo-ft.
Maybe not.

[ft] Remove (probably) stale comment

Minor

Install ot-funcs on newly created funcs

**Finally**!  Casual users can stop caring about font-funcs completely now,
like they haven't been needing to care re unicode-funcs for a few years.

[ot-font] Don't pre-load cmap table

Now that we have get_h_advances() and get_nominal_glyphs() implemented, the
overhead of doing a proper atomic load would be once per run, NOT once per
glyph.  So, no need to pre-load the tables to avoid that overhead.

As such, hb_ot_font_set_funcs() has become really cheap.  Can *finally* make
it be default font functions on all newly created fonts!

[ot/ft] Implement get_nominal_glyphs() callback

Some more measurable speedup.  The recent commits' speedups are as follows:

Testing with Roboto, ****when disabling kern and liga****:

Before:

FT --features=-kern,-liga
user↦   0m0.521s

OT --features=-liga,-kern
user↦   0m0.568s

After:

FT --features=-liga,-kern
user↦   0m0.428s

OT --features=-liga,-kern
user↦   0m0.470s

So, 17% speedup.

Note that FT callbacks are faster than OT these days since we added an advance
cache to FT.  I don't think the difference is enough to justify adding a cache
to OT.

When not disabling kern, the thing is three times slower, so the speedups
are three times less impressive...  Still, 5% not bad for a codebase that I
otherwise thought is optimized out.

Note that, because of this and other optimiztions in our main shaper,
disabling kern and liga, the OT shaper is now *faster* than the fallback
shaper.  So, that's my recommendation to clients that need the absolute
fastest...

Call get_nominal_glyphs() for runs of simple clusters at a time

Even without FT or OT font funcs implementing get_nominal_glyphs(), there's measurable
speedup.

Whitespace

Avoid sort and recompose stages if all clusters simple

Even has measurable speedup...

Fix warning

How come this one is not generated by clang everything bot?!

../../../test/api/test-multithread.c:37:26: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
 static char *font_path = "fonts/Inconsolata-Regular.abc.ttf";
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../../test/api/test-multithread.c:38:21: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]

Rewrite main normalizer loop to isolate runs of simple clusters

Inline decompose_cluster

Towards separating the common case into its own loop.

[font] Add get_nominal_glyphs() callback (note the plural)

Unused as of now.  To be wired up to normalizer, which would remove
overhead and allow hb-ot-font initialization to become a no-op, so
we can enable it by default.

Minor

test/api/test-subset-*.c: Fix build on pre-C99

Ensure that we have the variables at the beginning of the block.  These
are the only fixes that we need for building HarfBuzz on older
compilers.

builds: Fix and clean up MSVC DLL builds

Instead of passing a CFLAG/CXXFLAG to define HB_EXTERN, define it
directly in src/hb.hh as __declspec(dllexport) extern when we are
building HarfBuzz as DLLs on Visual Studio.  Define HB_INTERNAL
as nothing without defining HB_NO_VISIBILITY when building HarfBuzz as
DLLs to avoid linker errors on Visual Studio builds.

Also "install" harfbuzz-subset.dll into $(PREFIX)\bin as the
hb-subset utility will depend on that DLL at runtime, when HarfBuzz is
built as DLLs.  Since it consists of private APIs that are subject to
change, we do not install its headers nor .lib file.

Another old bot fix

Try fixing older bots

Add const to get_*_advances API

Ouch!

Port test off deprecated API

Minor

[icu] Unbreak

Remove test for deprecated hb_set_invert()

Properly remove deprecated stuff

Remove deprecated decompose_compatibility stuff

Remove deprecated eastasian_width

[ft/ot] Remove implementation of deprecated kerning funcs

[kern] Minor

[kern] Minor

Deprecate font kern API

Apply TrueType/OpenType kern table when GPOS kern feature is not available

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

Separate fallback kern vs mark positioning

[kern] Add kerning driver to TT kern table

[kern] Abstract away kerning machine

Add HB_DEPRECATED

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

[kerx] Process coverage flags

Deprecate decompose_compatibility stuff

Deprecate eastasian_width stuff

Minor

Minor

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

[kerx] Minor

[kern] Shout less

[kerx] Clean up Format2

[kern/kerx] Fix Format2 offsetting

"The values in the right class table are stored pre-multiplied by the
number of bytes in a single kerning value, and the values in the left
class table are stored pre-multiplied by the number of bytes in one
row. This eliminates needing to multiply the row and column values
together to determine the location of the kerning value. The array can
be indexed by doing the right- and left-hand class mappings, adding the
class values to the address of the array, and fetching the kerning
value to which the new address points."

[kerx] Clean up Format0

Support HBUINT32 BinSearchArrayOf

Move code

[aat] Rename

Rename invisible_codepoint to invisible_glyph in API

Deleted recently added API:
    hb_buffer_set_invisible_codepoint()
    hb_buffer_get_invisible_codepoint()

    hb-shape / hb-view --invisible-codepoint

New API:
    hb_buffer_set_invisible_glyph()
    hb_buffer_get_invisible_glyph()

    hb-shape / hb-view --invisible-glyph

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

Revert "[morx] Fix MORX-35"

This reverts commit f62f6e90ad1f1a83f77771ad65ee1ffb79470a8a.

[buffer] Improve shift_forward()

"Improve" is a strong word in this case though, I understand.

[morx] Fix MORX-35

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

Remove some code

We use scratch-flags to short-circuit this function.  No need for previous
early loop.

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

[kerx] Clean up kerx and KerxTable structures

Add API for setting invisible-codepoint

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

New API:
hb_buffer_set_invisible_codepoint()
hb_buffer_get_invisible_codepoint()

hb-shape / hb-view --invisible-codepoint

[ot-font] Load hmtx/vmtx lazily

Since we have get_*_advanes() API now, the overhead is once per shape,
not once per glyph.

Only cmap is warmed-up at set_funcs() time now.

Minor

Minor

[coretext] Fix OS X check

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

[morx] Fix memory access issues with unsafe_to_break

Make msan output a little more readable

Use tempfile in run-tests as a fix for Windows CI fails (#1228)

[uniscribe/coretext] Fix for previous change

[vector] Make hb_vector_t relocatable / nestable

Ugly, but...
Fixes https://github.com/harfbuzz/harfbuzz/issues/1227

Add hb_vector_t::fini_deep ()

[morx] Fix memory access issue

If buffer was enlarged, info was being outdated.

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

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

Change vendor features from Harf/Buzz to HARF/BUZZ

https://github.com/harfbuzz/harfbuzz/commit/a01194aaf4c15160330b4042066263b2c963b658#commitcomment-30772041

"The tag space of tags consisting of four uppercase letters (A-Z) with no punctuation,
spaces, or numbers, is reserved as a vendor space. Font vendors may use such tags to
identify private features."

More bot fixes

Fix a warning

[test] Choose 'ot' shaper specifically

Now that we added morx support, our OS X bot is running them through CoreText
and failing (with a DoS / infinite loop no less!).  Always run tests through
our own shaper.

[TODO] Clean up

Enable two OpenType features, 'Harf' and 'Buzz'

One early, before script-specific features, one late, after.  Allows font
developers to detect us and behave differently if needed.

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

Minor

Apply morx if there's no GSUB!

[aat] Add +hb_aat_layout_has_substitution

Minor: Fix autoconf warning

Before this change, autoconf was emitting the following warnings:

```
configure.ac:22: warning: AC_COMPILE_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_RUN_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_COMPILE_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_RUN_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_COMPILE_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_RUN_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_COMPILE_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
configure.ac:22: warning: AC_RUN_IFELSE was called before AC_USE_SYSTEM_EXTENSIONS
../../lib/autoconf/specific.m4:368: AC_USE_SYSTEM_EXTENSIONS is expanded from...
configure.ac:22: the top level
```

[morx] Only insert glyphs at mark when a mark has been set before

This reverts commit f4072e8cb81072cd6d51a2607efedb76c02e7db1.
https://github.com/harfbuzz/harfbuzz/issues/1195

Minor, fix double-promotion warnings (#1221)

State our graphite2 dependency no-deprecated compile flag (#1220)