This reverts commit face7cf5.

This reverts commit 7bf19801.

Fully untested.

MAP_NORESERVE is not available on macOS for example so set the flag
to zero if not defined on the headers.

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.

Makes for cleaner code.

* Adds libnsl to the bot as it is now needed apparently
* Disables hb-icu, apparently the newer ICU itself is not compatible with SunCC

Fixup for previous commit. Was accidentally turned off.

Make clang happy. Also, don't use anonymous union.

We'll see if this sticks to the bots.

"writable" is not in use and probably we can go in the reverse direction
and remove the flags that are useful for enabled writable, this is just
for the sake of completeness however.

It removes io.h and other polyfills which we no longer need as 7e76d746

This makes it compatible with ARMCC which I had access in
a collaboration with @imgtec, thanks!

Basically hb_blob_create_from_file features three code paths,
mmap, Win32 and fallback.

We had fallback implementation even before this but it was relied
to "open" which is not available on some environments. This change
improved the situtation by using only fopen and friends for
fallback path.

Interestingly we could use "open" on Windows but in fact it was
emulated by MSVCRT so I've completely split that from Unix path
now that we have a distinct path for fallback path also.

Seems like I messed up; buffer overrun got reported.

Following to 63f57f4d

Trying to see how many can converted meaningfully.