Tighten Coverage iteration

[subset] Minor

[iter] More semicolon

[iter] Use aliasing using for types

Fix ambiguity of hb_sorted_array_t::item_t with gcc.  No idea if that's a gcc bug
or what spec requires, but using aliasing using seems to fix it.  It probably breaks
our non-C++11 bots, in which case I have to condition the change.  Testing.

[iter] Use static_assert with hb_is_random_access_iterator()

Both, checks constexpr'ness, and fixes build with cra**y implementations
of assert() macro:

test-iter.cc:108:11: error: too many arguments provided to function-like macro invocation
  assert (hb_is_random_access_iterator (array_iter_t<int>));
          ^
./hb-iter.hh:186:42: note: expanded from macro 'hb_is_random_access_iterator'
  hb_is_random_access_iterator_of (Iter, typename Iter::item_t)
                                         ^
/usr/include/x86_64-linux-gnu/sys/cdefs.h:89:9: note: macro '__STRING' defined here
\#define __STRING(x)     #x
        ^
test-iter.cc:108:3: error: use of undeclared identifier '__STRING'
  assert (hb_is_random_access_iterator (array_iter_t<int>));
  ^
/usr/include/assert.h:91:21: note: expanded from macro 'assert'
   : __assert_fail (__STRING(expr), __FILE__, __LINE__, __ASSERT_FUNCTION))
                    ^

[iter] Remove stray semicolons

[OT] Implement operator[] for Coverage and ClassDef

[iter] Add TODO

[iter] Implement for OT::ArrayOf / OT::SortedArrayOf

[SortedArrayOf] Fix sub_array() return type

[iter] Constrain hb_fill() and hb_copy()

[iter] Add TODO

[iter] Add hb_is_random_access_iterator()

[iter] Rename hb_is_[sorted_]iterator() -> hb_is_[sorted_]iterator_of()

[meta] Add hb_declval() macro

[iter] Syntax

[iter] Test hb_is_iterable / hb_is_iterator

[serialize] Make SortedArrayOf:;serialize() take sorted-iterator

[iter] Fix operator->

[array] SFINAE fun

[meta] Rename

[array] Remove construction that was removing constness

[iter] Fix test

[iter] Remove comment

[iter] Use is_sorted_iterator

[iter] Handhold hb_is_iterator() type deduction

by partial-instantiating on Iter.

Revert "[iter] Add hb_iter_of_t<>"

This reverts commit d6cbe96e2fc7bc8f1c10e631b52b1ef31ff9a6f5.

Isn't useful, as duplicate inheritance of same type results in ambiguity
errors...

[iter] Fix warnings

[iter] Add hb_iter_of_t<>

[iter] Remove hb_sorted_iter_t

Not enforcing it using type hierarchy.

[iter] Const correctness

Add operator= to IntType, commented out

https://github.com/harfbuzz/harfbuzz/pull/1510

[iter] Minor

[iter] Make is_random_access_iterator a constant

We cannot rely on constexpr functions...

Separate GlyphID from HBUINT16

For stricter enforcement.

[iter] Fix sorted_iter iter class

[iter/meta] Add hb_is_sorted_iterator()

[iter] Remove stale comment

[iter] Port Coverage towards iter_t instead of array_t specifics

[meta] Move typename around

We'll see if bots like.

[meta] Move code around

[iter] WHitespace

[iter] Port Coverage::serialize to hb_is_iterator

[iter/meta] Match hb_is_iterator<> using SFINAE

By specifying Item type, which is desirable.

Minor

Use C++11 when available

[ci] Remove unused config for gcc 4.2

[skip ci]

[ci] Disable other gcc 4.2 bot

[iter] Add back operator +

Too ugly to remove..

[iter] Disable operator +

To see if it makes bots happy... This is frustrating.

[iter] Revert back uses of C++11 auto type deduction

[iter] Rename

[iter] Remove friend operator +, hoping to fix some bots

Use Null() instead of declval(), hoping to fix some bots

[meta] Don't use template default arguments for functions

That's a C++11 extension apparently...

[meta] Fix unused-function-template warning

[iter/meta] Implement is_iterator

Removes use of auto type deduction again, which was not supported on many bots.

[meta] Minor

[meta] Move more code here

[iter/meta] Add hb_is_iterable

Don't use delegating constructors

Not all C++11 features are created equal when it comes to support...

[meta] Add enable_if

Add hb-meta.hh for meta-programming

[iter] Clarify readonly vs lvalue iterators

lvalue iterators must declare __item_type__ as a reference.

[iter] Test OT::Coverage iter

[iter] Streamline vector iterators

[iter] Make hb_sorted_array_t work as iter

Ugly, but does the job.

Change hb_void_t implementation

[iter] Mark Coverage iterator sorted

[subset] Take iterator in ArrayOf serialize

Still not satisfied with how I can enforce iterators only, but
seems to work for now.

[iter] Port Coverage iterator to hb_iter_t

Add hb_pair_t<> and hb_pair()

[Coverage] Ensure increasing coverage in iteration

[Coverage] Mark iterator methods const

[iter] Use operator bool in a few places

[iter] Remove redundant methods

[iter] Test default-constructability

[iter] Implement friend opeator + (int, iter)

[iter] Remove const_iter

[ci] Disable macos-llvm-gcc-4.2 again

Not C++11.

[iter] Remove hack for older compilers

[iter] Minor

[iter] Rename random_access() to constexpr is_random_access()

[iter] Add operator ->

[iter] Make them work, mostly

[vector] Add iterator

Remove wrongly added files

Use enum for class constant

[CFF] Use enum for tableTag

Use static constexpr for large class constants

Fix sign compare warnings

../../third_party/harfbuzz-ng/src/src/hb-map.hh(56,45):  warning: comparison of integers of different signs: 'const hb_codepoint_t' (aka 'const unsigned int') and 'hb_map_t::(anonymous enum at ../../third_party/harfbuzz-ng/src/src/hb-map.hh:169:3)' [-Wsign-compare]
    bool is_unused () const    { return key == INVALID; }

Fix warning

warning: '_WIN64' is not defined, evaluates to 0 [-Wundef]

[dwrite] Fix delete-non-virtual-dtor warning (#1550)

Remove TRACE_COLLECT_GLYPHS

[pragma] Only add if GCC or clang

[directwrite] More fix

[dwrite] Try to fix delete-non-virtual-dtor warnings

Fixes #1548 hopefully

Remove tracing from closure

Fix -Wundef errors with __GNUC__

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

init interp_env_t::hintmask_size

fixes issue #1547