They are not sorted.

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

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

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.

Okay, let's see if the gods are happy now...

Ugly as hell, and don't even understand why some bits are needed.
But the logic is sound.

Sorry for the noise, downstream custom builders.  Please adjust.

Also remove "    " null data for Tag.  Just use zeroes.

Should fix Windows build again.

Move out-of-class definitions of two methods to hb-static so they
are accessible in libharfbuzz-subset.

This was causing problem on systems without visibility when map was used
from both libharfbuzz and libharfbuzz-subset. Sigh.

https://ci.appveyor.com/project/harfbuzz/harfbuzz/build/1.0.1669/job/dey47nmff0770vp3

Clang build was failing with:

/usr/bin/ld: .libs/libharfbuzz_la-hb-blob.o: relocation R_X86_64_PC32 against undefined hidden symbol `_ZTH12_hb_CrapPool' can not be used when making a shared object

Instead of fighting it, just disable use of it.

Part of fixing https://github.com/behdad/harfbuzz/issues/585

This reverts commit 80af2812, reversing
changes made to c442672e.