[iter] Sketch new iterator design

[iter] Add empty test

Revert "Remove unused hb-iter.hh"

This reverts commit 969ff3c7aadbe721cdd414488eb170433f10d00c.

[vector] Add operator bool

Rename

Remove unused typedef

2.3.0

[serializer] Add operator <<

Not sure if we are going to use it.  But might incentivize us to.

Add hb_assign(obj, value)

[serialize] Adjust ClassDef

[skip ci]

Fix code on big-endian gcc / clang

Ouch!  We need a bigendian bot...

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

Minor [skip ci]

[saitnize] Minor

Fix build on older Visual Studio versions (#1499)

* src/hb-cff-interp-dict-common.hh: Use ull for unsigned int64_t

The llu suffix does not work for older Visual Studio versions
(pre-2013), but ull works for all the compilers that we attempt to
support.

* test/api: Fix build on pre-C99 compilers

Ensure variables are declared at the top of the block.

* src/hb-dsalgs.hh: Add specialization for hb_is_signed<> for __int8

Pre-Visual Studio 2010 does not consider __int8 (which is typedef'ed to
int8_t) to be equivilant to signed char, so the compiler cannot find the
corresponding hb_is_signed<> specialization that is needed.

The interesting thing is unsigned __int8 is considered to be equivilant
to unsigned char, so as the other types (short, int, long) that we look
for here, so only the specialization for __int8 is added here.

This will fix builds on Visual Studio 2008 at least.

[sanitize] Use hb_static_size instead of ::static_size

https://github.com/harfbuzz/harfbuzz/issues/1496#issuecomment-448818112

[test] Fix test-name-table.cc leak issue

Spotted it accidentally but wanted to see if is anything serious so went for fixing it

[arrays] Add hb_ubytes_t for unsigned char

[arrays] Remove hb_supplier_t<>

[serialize] Propagate error from hb-face

[serialize] Streamline error propagation

Remove unused hb-iter.hh

The ideas there are all part of hb-array.hh now.  To be determined how we
want to use generic iterator patterns.

[arrays] Minor tweaks to hb_supplier_t

I think I like to keep this hb_supplier_t thing separately from hb_array_t.

[arrays] Rename Supplier to hb_supplier_t

Hide hb_addressof()

Enable __builtin_* on clang

We'll see which old clang versions this breaks...

Add hb_addressof() and use it to fix bug after hb_bytes_t merge

We cannot take address hb_bytes_t direction.  We need to use the
newly added hb_addressof(), ala std::addressof().

Enable system extensions in hb.hh

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

Fix Solaris use after free (#1495)

Remove define GNU_SOURCE

Not needed.  We get it in our config.h automatically thanks to
AC_USE_SYSTEM_EXTENSIONS.  Let's see whose build it breaks...
If we end up putting it back, we should add other things from
that macro and remove the macro.

Fix more warnings

Fix Solaris Trailing comma in enum warnings (#1490)

Lower coretext_aat shaper priority (#1488)

Related to https://github.com/harfbuzz/harfbuzz/issues/1478

Remove redundant void from C++ sources (#1486)

Minor, fix two more ArrayOf incorrect operator logic

[arrays] Minor fix

[array] Add cast operator to add const to Type

In lieu of constructor removed in previous commit.

[array] Remove problematic constructor

[array] Add arithmetic operators

[array] Return Crap instead of Null if Type is not const

Ouch!

[array] Add operator *

[array] Add constructor from fixed-size array

[array] Organize

[arrays] Reduce Supplier<> even further

Whitespace

[array] Add operator +=

[array] Add constructor from hb_array_t<const Type>

[arrays] Move Supplier<> to hb-array.hh

[arrays] Remove unnecessary constructor from Supplier<>

Looks like operator hb_array_t<> from vector works here. :)

[arrays] Start moving Supplier<> to hb_array_t<>

[arrays] Remove unused stride from Supplier

[arrays] Remove need of stride in Supplier<>

[array] Remove custom hb_bytes_t implementation

[arrays] Minor

[array] Minor

[array] Move hb_array_t and related types to hb-array.hh

Minor change to explicit_operator aesthetics

Remove redundant 'inline' from methods (#1483)

Whitespace

Whitespace

[subset] Actually subset GDEF

[subset] Fix up ClassDef some more

[subset] Fix ClassDefFormat1 subsetting

[subset] Sketch GDEF subsetting

[subset] Implement for ClassDef

Minor

[serialize] Implement for ClassDef

[serialize] Implement for ClassDefFormat2

[serialize] Implement for ClassDefFormat1

[subset] Minor

[CFF] bad offset in Index (#1476)

* Update hb-ot-cff-common.hh

* fix bug

* bummer fix wasn't hit. refix

* additional sanity check

* Added test cases for oss-fuzz issues 11805, 11806

[CFF] check out of range FD index (#1477)

* add fd index checks to subr subsetter

also added oss-fuzz test case

* undid SubrSubsetParam::is_valid

because already validated by SubrClosures.valid

[util/hb-subset] Add --layout to keep GDEF/GSUB/GPOS

Will become default and option removed in the future.

[dispatch] Minor

[sanitize] Fix sanitizing sublookup array

[dispatch] Minor

Revert "Minor fix re sanitize of Lookup subtables"

This reverts commit 7146718bef81492e13aede0a2801cda1da41ce35.

Fixing differently.

minor style fix, use void in methods on no argument

Merge pull request #1474 from fanc999/master.msvc

Few fixes for Visual Studio builds

Minor fix re sanitize of Lookup subtables

We were dereferencing Null pointers and trying to sanitize them,
which is not necessary...

Minor

[subset] Map glyphs during SingleSubst subsetting

Ha!

[subset] Minor

[serialize] Break down assert

Minor

[CFF] oss-fuzz issue 11690 ASSERT: substr.offset >= opStart (#1461)

* fix oss-fuzz 11690: substr.offset >= opStart

detect recursive subroutine call & handle as error

* fix build failure

* add minimized test case for oss-fuzz 11690

* removed asserts

minimize use of assert: removed or changed to error handling (#1467)

[ci] Fix Travis macOS bot ICU issue (#1472)

[ucdn] Fix header

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

Correct REPLACEMENT CHARACTER's code point to FFFD (#1471)

test/api: Fix building on pre-C99 compilers

Ensure variables are declared at the top of the block.

CMake: Fix Introspection builds

We need to add -DHB_AAT_H and -DHB_AAT_H_IN to the flags that are passed
to g-ir-scanner, so that introspection builds can proceed normally.

[glyf] minor

[os2] Move typometrics detection logic to the table (#1465)

Apply non-controversial parts of ot-style (#1464)

Things to be used in https://github.com/harfbuzz/harfbuzz/pull/1459

fix for issue #1447 (#1462)

Added case for OpCode_BaseFontName. This opcode in spec but practically unused.
Added a comment for default case which can't be hit

[fdsc] minor

[CFF] Refix oss-fuzz 11714: set_blends (PR #1458) (#1460)

* pass subarray of stack to set_blends

* get_subarray to return a value, not ref

* restored error check (with tweak)

Fix likely check

Ouch!

Add default value to first argument of sub_array()

Add sub_array to hb_vector_t