Fails locally.  Trying to understand.  Sigh

Apparently there's different overload resolution rules that apply, at
least with some (older?) version of gcc.

hb-ot-name-table.hh: In member function ‘void OT::name::accelerator_t::init(hb_face_t*)’:
hb-ot-name-table.hh:244:62: error: ambiguous overload for ‘operator+’ (operand types are ‘hb_blob_ptr_t<OT::name>’ and ‘OT::NNOffsetTo<OT::UnsizedArrayOf<OT::IntType<unsigned char, 1u> > > {aka const OT::OffsetTo<OT::UnsizedArrayOf<OT::IntType<unsigned char, 1u> >, OT::IntType<short unsigned int, 2u>, false>}’)
       this->pool = (const char *) (const void *) (this->table+this->table->stringOffset);
                                                              ^
hb-ot-name-table.hh:244:62: note: candidates are:
hb-ot-name-table.hh:244:62: note: operator+(const C*, long int) <built-in>
hb-ot-name-table.hh:244:62: note: operator+(const char*, long int) <built-in>

This reverts commit 01912efb.

Actually this didn't break things.  Fixing

Reverts 57f65ae9

Caused ambiguous-overload on some gcc...

See comments.

We should add a bot for it
Part of #1652

Fixes an unused function warning when building with HB_NO_SUBSET_LAYOUT
as part of the Chrome build.

https://github.com/grigorig/ucdn/issues/23

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

No idea where these appear from...

This make output of hb_enumerate() sorted regardless of input iterator.

More correct behavior.  We were commenting out some legit conditions
before.

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

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

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

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

This reverts commit 9b05db33.

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

This reverts commit ae8719eb.

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

Fedora upgraded to ragel 7, which is buggy if char is signed.
Switching to -G2 output fails with sign-compare error:

../../src/hb-buffer-deserialize-json.hh:107:12: error: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘const char’ [-Werror=sign-compare]
    if ( 9u <= ( (*( p))) && ( (*( p))) <= 13u ) {
         ~~~^~~~~~~~~~~~~

Switching to -T1 for now.  It actually results in smaller code,
at the expense of some binary searching instead of flat tables.
In the not distant future, we might actually generate two different
outputs and choose between depending on size-optimize options.

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