[indic/khmer/myanmar/use] Clarify clear_syllable

No logic change.

[fuzz] Add a new testcase

[layout] Try to speed up collect_lookups some more

Barely made a dent :(.

[layout] Memoize collect_features

Fixes https://github.com/harfbuzz/harfbuzz/pull/1317
Fixes https://oss-fuzz.com/v2/testcase-detail/6543700493598720

[layout] Remove unintentionally added code

[layout] Add hb_collect_features_context_t

Towards https://github.com/harfbuzz/harfbuzz/pull/1317

[lookup] Fold another function inline

[layout] More prep work to memoize collect_features() work

[lookup] More prep work for memoizing collect_features

https://github.com/harfbuzz/harfbuzz/pull/1317

[layout] Minor

We were returning the accelerator's lookup count.  Returns table's.
They are the same except for OOM cases.  Just shorter code.

[layout] Fold one function inline

Preparation for fixing https://github.com/harfbuzz/harfbuzz/pull/1317

[morx] Use deleted-glyph, instead of actual deletion, in Ligation

Closer to what CoreText does.

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

[morx] Remove deleted-glyph at the end of processing

[aat] Minor

[aat] Start adding support for DELETED-GLYPH

https://github.com/harfbuzz/harfbuzz/issues/1302

[vector] Make it act more like pointer

Add pointer cast operator and plus operator.

[set] Minor

[set] Make array access more explicit

Follow up on 94e421abbfc7ede9aaf3c8d86bb0ff9992ea3123

Fix again

[docs] Add hb-ot-color section

Fixup

[test] Rename some fonts

If we have duplicae font files in different directories, that would
break the oss-fuzz build currently.  So, rename some to avoid
name class with text-rendering-test.  Would be better to find
another solution.

Fix possible overflow in bsearch impls

From bungeman.

Fixes https://github.com/harfbuzz/harfbuzz/pull/1314

Remove some use of Crap in hb-set.

When hb_set_t::page_for_insert needs to insert at the end of the page_map
it ends up evaluating '&page_map[i + 1]' which has hb_vector return an
lvalue of a Crap so that nothing can be moved to its address. This turns
into issues with ThreadSanitizer on Crap when two threads modify different
hb_set_t instances. This can be avoided by using '&page_map[i] + 1'
instead.

Set up CI with Azure Pipelines

[appveyor] Drop Cygwin builds again

They are so slow and we had only a couple of Cygwin build failure
reported in ~5 years.

[appveyor] Limit make to three jobs

https://github.com/harfbuzz/harfbuzz/pull/1309#issuecomment-432778270

[vowel-constraints] Reset continuation on all dottedcircles

One of the was missed before.  Not intentional.

[appveyor] Build on Cygwin and msys2 in parallel

[appveyor] Do not update msys2

Looks like AppVeyor has new enough packages for our needs. Cuts CI build
time by 5 minutes.

[appveyor] Organize config file a bit

[appveyor] Don’t build branches with open PR twice

[ot-font] Fix sign of ascent/descent

Some fonts, like msmincho, have positive number as descent
in vhea table.  That's wrong.  Just enforce sign on ascent/descent
when reading both horizontal and vertical metrics.

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

[morx] Fix cluster-merging when ligating

Only merge when actual ligature happened.

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

Fix size calculation in DEFINE_SIZE_ARRAY_SIZED.

[aat] Fix up previous commit and add files

[aat] Allow enabling/disabling features

Only works at entire-buffer range, not sub-ranges.

Test with:

$ hb-shape Zapfino.dfont Zapfino
[Z_a_p_f_i_n_o=0+2333]

$ hb-shape Zapfino.dfont Zapfino --features=-dlig
[Z=0+416|a=1@-21,0+264|p_f=2+433|i=4+181|n=5+261|o=6+250]

$ hb-shape Zapfino.dfont Zapfino --features=+dlig
[Z_a_p_f_i_n_o=0+2333]

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

[ot-map] Minor

Minor

Shift code around

[trak] Allow disabling tracking for ranges of text

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

When parsing feature ranges, accept ';' instead of ':'

Make tracking optout possible using 'trak' ot feature

So '--features=-trak' or 'font-feature-settings: 'trak' 0;' can prevent
applying of tracking if used.

Touch up on previous commit

https://github.com/harfbuzz/harfbuzz/pull/1273

[use] Prohibit visually ambiguous vowel sequences

[VORG] Hook up to hb-ot-font's v_origin

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

Test added with NotoSansCJK, eg. with U+FF38.

[VORG] Add get_y_origin()

Unhooked.

[VORG] Add VORG table

Cherry-picked from cff-subset branch.

[morx] Add a few debug messages to Ligature chain

[morx] Clear stack upon underflow when acting ligatures

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

[morx] Leave ligature on stack after ligating

Test with Apple Color Emoji.ttf of ~66MB and following sequence:

./hb-shape Apple\ Color\ Emoji-old.ttf --font-funcs=ot -u U+1F468,200d,U+1F469,200d,U+1F467,200d,1f466

Should form full family if matching works correctly.  It first makes
family of three, then makes family of four out of that and the last
two characters.

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

[ci] Disable llvm-gcc

  #define Null(Type) Null<typename hb_remove_const<typename hb_remove_reference<Type>::value>::value>()

raises:
  hb-blob.cc: In function 'hb_blob_t* hb_blob_get_empty()':
  hb-blob.cc:194: error: using 'typename' outside of template
  hb-blob.cc:194: error: using 'typename' outside of template

Removing "typename"s fixes the issue but makes newer compiler fail apparently.

Probably downstream can patch that locally till we get a solution.

Unbreak builds

Add hb_array<>() specialization for UnsizedArrayOf

Related https://github.com/harfbuzz/harfbuzz/issues/1301

Add hb_array<>()

Simplifies transient object creation.

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

[color] Use Index for colorIdx

Doesn't matter, but matches the description.

Remove const and references when binding Null()

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

Removes anomaly I was seeing in cpal table trying to use implicit Null(NameID).

[color] Add HB_COLOR

Normally I would have added to_string / from_string like other types
have.  But since we don't use hb_color_t much, I'm not going to do that.
Although, if we did, we could use it in hb-view to parse colors...

[color] Finish reviewing / revamping CPAL

Now to hb_color_t.

[color] Minor

[color] Rely on CPALV1Tail Null object

Remove const from hb_array_t details

Will come in through <T> if desired.

[color] Use hb_array_t in CPAL

Doesn't work though, ouch :(.  Need to figure out if it's unreasonable
to expect Null(T) inside hb_array_t<T> to see the later specialization
of Null for NameID.

[color] Check for null CPAL arrays

We cannot use a nullable offset here though.

[color] More CPAL rename

[color] More CPAL rename

[color] Rename vars in CPAL

Adopt dump-emoji with latest unreleased APIs changes (#1297)

[color] Revamp palette API

[color] Rewrite colr access

COLR table has one function: return layers for a glyph, and we expose exactly
that, so should just wire it through.  Also use sub_array() for verifiable
safety.

Also, BaseGlyphRecord's null object is enough.  We don't need to special-case
the not-found.

[colr] Add hb_ot_color_layer_t

We never return parallel arrays from functions.  That's inconvenient
and hard to bind.

[color] Rename / reorder a bit

Implement has_data() for realz.

[cpal] Touch up palette flags

[color] Rename "gid" to "glyph"

We don't expose "gid" in API.

[colr] Move sanitize

Add chromacheck Emoji test fonts and minor preparations (#1294)

[colr] Move compare function into a static

Not sure if MSVC would be unhappy about this.

[colr] Minor

[colr] Move sanitize() to right place

Sanitize always comes just before data member definitions, so
it's easy to cross-check.

[colr] Touch up a bit

When a struct is plain old data with no references, etc, it's okay
to mark its members public.

[color] Fix documentation a bit

Minor

Remove _palette_entry_count as can be done with _palette_colors

[test] Test hb_ot_color_has_{colr,cpal}_data

[CPAL] Add palette entry and enable palette flag API

Add API test for hb_ot_color_get_color_layers

Address COLR/CPAL reviews and revive cpal_v1 tests

[color] Fix alpha channel value and adjust spaces

[dump-emoji] Formatting

[test] Use hb_test_open_font_file

Add three macros for separating color channels

Make ot-color tests pass

[color] Minimal API for COLR/CPAL

[fuzz] Add a found hb-subset testcase

[ci] Disable flaky -windows-x64 and add a comment for iOS

[fuzz] Add more testcases

Fixed already but better to have anyway.

One didn't have minimized but it was only 164 B, so

Merge pull request #1290 from ebraminio/testopenfont

[test] Unify font file opening across the tests

[test] Unify font file opening across the tests

[fuzzing] Add more font