no-desubroutinize option is disabled for now
code cleanup (esp. CFF1 width handling)
bug fixes & renaming

along with test & font

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.

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

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

One of the was missed before.  Not intentional.

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

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

Only merge when actual ligature happened.

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

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

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

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

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

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

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

Unhooked.

Cherry-picked from cff-subset branch.

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

  #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.

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

Simplifies transient object creation.

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

Doesn't matter, but matches the description.

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

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