So, our fallback static_assert cannot be had more than once per line
of source.

Bots not happy with using "this" inside assertion...

This reverts 26566448

Still does that if get_null() returns nullptr.  Our shaper data objects
are like that.  Shrug.

Fixes https://oss-fuzz.com/v2/testcase-detail/5754863779053568

Ouch :).

Follow up on b4c61130

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

Newer compilers / language allows structs with constructor in union.
So, this was not actually testing anything.  Indeed, the recent
change in DISALLOW_COPY *is* making some of our types non-POD.
That broke some bots.

Just remove this since it wasn't doing much, and I'd rather have
DISALLOW_COPY.

Show up with gcc -O0.

There's a few more but those are functions that need to be filled in.

Maybe this is a lost battle...

Add pointer cast operator and plus operator.

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11034

Was givin a dozen of:

../../src/hb-machinery.hh: In member function ‘bool AAT::ankr::sanitize(hb_sanitize_context_t*) const’:
../../src/hb-machinery.hh:307:23: warning: missed loop optimization, the loop counter may overflow [-Wunsafe-loop-optimizations]
     bool ok = --this->max_ops > 0 &&
               ~~~~~~~~~~~~~~~~~~~~~~
        this->start <= p &&
        ~~~~~~~~~~~~~~~~~~~
        p <= this->end &&
        ~~~~~~~~~~~~~~~^~
        (unsigned int) (this->end - p) >= len;
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I believe those are bogus, but this silences them and does not introduce
logic issues I believe.

For 329f2401

max_ops is signed.

Since we consume it legitimately during shaping.

Ugly, but...
Fixes https://github.com/harfbuzz/harfbuzz/issues/1227

Some clang versions define static_assert as a macro apparently, so we cannot
redefine it...

This reverts commit 94bfea0c.
This reverts commit 4e626278.