Prevents O(n^2) run times.

This new API returns cvXX and ssXX related NameId, things like
featUiLabelNameId, featUiTooltipTextNameId, sampleTextNameId, ... of cvXX
and UINameId of ssXX, in a unified way.

However HarfBuzz currently doesn't expose an API for retrieving the actual
information associated with NameId from the `name` table and that should be
done separately.

Also, uncomment code again, just "if (0)" it out, so it doesn't get stale again.

Towards possibly using Null pool for some nil objects.

If a pointer type was passed to Null(), reinterpret_cast<> was
complaining about qualifiers being removed. Turns out I need the const on
the other side of "Type" to fix that.  Also remove unused const from
NullPool type.

Not sure if I like to keep it. For now, aim for correctness.

Common Regoin for Access Protection.  Like the NullPool, but writable.

We should hang those off somewhere else. For now, the unused ones can go.

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

If we don't have visibility, make NullPool static.

If the `calloc` for `gsub_accels` or `gpos_accels` fails, then the
unlikely branch afterwards can be taken, which frees up the
`hb_ot_layout_t`, but since those fields can now be `nullptr`, then
we don't want to dereference them.

Untested, but compiles.

(And prevent it from happening in the future.)

Saves some more code size as well!

Saves 2k of .bss section.