[ci] Enable more of GCC compile warnings (#1533)

Usermanual: small formatting and wording fixes.

Usermanual: small updates.

CircleCI: Test for Python 3 syntax errors and undefined names (#1522)

Catch missing imports and errors like #1520 and #1521

__E901,E999,F821,F822,F823__ are the "_showstopper_" [flake8](http://flake8.pycqa.org) issues that can halt the runtime with a SyntaxError, NameError, etc. Most other flake8 issues are merely "style violations" -- useful for readability but they do not effect runtime safety.
* F821: undefined name `name`
* F822: undefined name `name` in `__all__`
* F823: local variable name referenced before assignment
* E901: SyntaxError or IndentationError
* E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree

Fix automake warnings

[skip ci]

print() is a function in Python 3 (#1520)

Python 3 fixes to gen-os2-unicode-ranges.py (#1521)

In Python 3, __reload()__ was moved and __sys.setdefaultencoding()__ because the default is already utf-8.  Also __Error()__ is an _undefined name_ and __Exception()__ creates a generic exception.

[cmake] Enable C++11 on CMake (#1519)

Ouch! Fix build on C++<11

Fix ubsan with passing nullptr to qsort()

Fix build on C++ < 11

[vector] Fix warning

[aat] Minor

[vector] Remove use of arrayZ() by using casts

[vector] Remove static_array

Was good idea, but with C++ types with constructor/destructor, was getting in
the way as compiler was destructing those items where it was not desired.
Since C++ does not allow zero-sized arrays, just remove it...

[vector] Use allocated = -1 to signify failure

fix lealk with cff2::accelerator_templ_t::topDict (#1517)

[vector] Change pre-alloced count from 8 to 2

I'm thinking about dropping it to zero, but that needs slight code changes.

Fix a few warnings

[iter] Fix warning

[set] Add iter_t as alias to const_iter_t

[set] Change to const_iter

[set] Cache length in iterator

Minor, remove redundant inline

Minor, tweak spaces

CFF renaming (#1507)

* reimplement ByteStr as byte_str_t based on hb_ubytes_t

Unuse start_embed<ByteStr>
Also renamed SubByteStr to byte_str_ref_t
More renaming to come

* substr renamed to str_ref in line with its type byte_str_ref_t

* uncamelize non-table struct names

* uncamelized non-struct types OpCode etc

* add byte_str_t copy ctor

* test

* test2

* undo tests

* fix bot failure

* undo the previous change

* fixed tabs, added inline

* Revert "fixed tabs, added inline"

This reverts commit 21163c30e9d18759414f7fe2518628241599f039.

* fix tabs

[set] Actually derive iterator from hb_sorted_iter_t<>

[set] Implement unified iterator

[set] Mark some internals protected

[iter] Add hb_sorted_iter_t<>

[iter] Change operator[] operand to be signed

To fix older compilers again (this was the case in hb_array_t).

hb-ot-layout-common.hh:1353: note: candidate 2: operator[](T*, int) <built-in>
hb-ot-layout-common.hh:1354: error: ISO C++ says that these are ambiguous, even though the worst conversion for the first is better than the worst conversion for the second:
hb-iter.hh:63: note: candidate 1: Item& hb_iter_t<Iter, Item>::operator[](unsigned int) const [with Iter = hb_array_t<const OT::IntType<short unsigned int, 2u> >, Item = const OT::IntType<short unsigned int, 2u>]
hb-ot-layout-common.hh:1354: note: candidate 2: operator[](T*, int) <built-in>
hb-ot-layout-common.hh: In member function 'bool OT::ClassDef::serialize(hb_serialize_context_t*, hb_array_t<const OT::IntType<short unsigned int, 2u> >, hb_array_t<const OT::IntType<short unsigned int, 2u> >)':
hb-ot-layout-common.hh:1490: error: ISO C++ says that these are ambiguous, even though the worst conversion for the first is better than the worst conversion for the second:
hb-iter.hh:63: note: candidate 1: Item& hb_iter_t<Iter, Item>::operator[](unsigned int) const [with Iter = hb_array_t<const OT::IntType<short unsigned int, 2u> >, Item = const OT::IntType<short unsigned int, 2u>]
hb-ot-layout-common.hh:1490: note: candidate 2: operator[](T*, int) <built-in>

[array] Port to hb_iter_t<>

hb_array_t is its own iterator...

[iter] Change __more__ to fallback to __len__

[array/vector] Rename len to length

[iter] Split hb_iter_t<> into hb_iter_t<> and hb_iter_mixin_t<>

Adjust internal header dependencies

[Makefile.am] Minor

[iter] Include from hb.hh

[iter] Move hb_fill() and hb_copy() to hb-iter.hh

[iter] Disallow copy-construction or assignment of hb_iter_t<>

Should only be done by subclass.

[iter] Fix hb_copy() return value

[iter] Add const_iter_t / const_iter()

[iter] Make hb_fill() take collection type, not iter

Starting to get the hang of when take which.

[array] Add more hb_array() / hb_sorted_array() variants

[serialize] Use component_list by reference, as a supplier

[iter] Add .random_access()

[iter] Fix __end__()

[array] Remove copy constructor

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

Fix bot

  C:\projects\harfbuzz\src\hb-uniscribe.cc(709): error C2666: 'hb_vector_t<OPENTYPE_FEATURE_RECORD,8>::operator +': 3 overloads have similar conversions [C:\projects\harfbuzz\build\harfbuzz.vcxproj]

Minor

[iter] Fix test again

Minor

[iter] More prototyping

[iter] Make hb_fill() and hb_copy() take iterators

I'm still going back and force...

[iter] Add hb_fill()

[iter] Start prototyping hb_copy()

[iter] Showcase implicit casts

Rename

[iter] Rename

[iter] First sample use

[iter] Add bidirectionality

[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!