Not sure why only one of the bots sees this as visible...

Maybe I should disable C++11 detection such that my compiler also catches
these.  Doesn't look like we can switch to it for a while anyway...

It's dead, Jim!

Apparently CoreText does allow these.  To be done: detect infinite
loops.

Fixes MORX-12 test.

Don't know why this is not caught by my compiler.  Looks like clang catches it
while gcc doesn't.

In file included from ../src/hb-aat-layout-morx-table.hh:31:
../src/hb-aat-layout-common-private.hh:523:30: error: no member named 'static_size' in 'AAT::ContextualSubtable<AAT::MorxTypes>::EntryData'
  DEFINE_SIZE_STATIC (4 + T::static_size);
                            ~~~^

Fixes MORX-11 test.

Tested with Zapfino and text "2nd". Sascha will create tests later.

Passes MORX-10 test now.

Neato.

To be improved later.

Such a headache that Windows defines UINT8, ...; Just prefix it.

c:\projects\harfbuzz\src\hb-aat-layout-common-private.hh(51): error C2872: 'UINT16': ambiguous symbol [C:\projects\harfbuzz\build\harfbuzz.vcxproj]
C:\Program Files (x86)\Windows Kits\10\Include\10.0.14393.0\shared\basetsd.h(80): note: could be 'unsigned short UINT16'
c:\projects\harfbuzz\src\hb-open-type-private.hh(648): note: or       'OT::UINT16'

Also makes hb-ot-shape call morx for now instead of GSUB... Just for testing.

Untested, but compiles.