[ci] Remove unnecessary warning declarations

clang -Weverything still left to be ported to pragmas

[pragma] ignored -Wunsafe-loop-optimizations // TODO fix

[pragma] error -Wclass-memaccess

[pragma] More

[pragma] More

[pragma] Flesh out more

Use NNOffsetTo<> in rest of places

Use NNOffsetTo<>

Add NNOffsetTo<>

[AAT] Comment

Comment

[AAT] Remove unused code

[AAT] Allow null in ankr lookupTable offset

[pragma GCC] Ignore -Wtype-limits

[ci] Remove -Werror from Travis

hb-machinery.hh:111:37: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]

Also ignore -Wtype-limits.

More GCC pragmas

Rename macro

Add more GCC warning pragmas

Move -Wcast-align to hb.hh

Add pragram GCC diagnostic ignored "-Wunused-macros"

[AAT] Fully sanitize ankr table at sanitize time

Third try to fix access. Followup 6879efc2c1596d11a6a6ad296f80063b558d5e0f

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

[AAT] Fix anchor bound checking, again

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

[aat] Minor

Improve overflow avoidance

Better fix for 480406cd3ef9e5ab8476ddfa04498bf23906c508
This way we behave the same on 32bit and 64bit archs.

fix FDSelect fuzzing bug (#1539)

Rewrote struct FDSelect3_4.ranges as ArrayOf
Updated FDSelect3_4::sanitize () to call ranges.sanitize ()
nRanges now a function to return a reference to ranges.len

Move _POSIX_SOURCE definition

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

[test] Add test for previous commit

Fix assertion on address overflow

Fixes https://bugs.chromium.org/p/chromium/issues/detail?id=917031

[AAT] Fix mort ContextualSubtable offset access

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

[AAT] Fix ankr table access

Fixes https://bugs.chromium.org/p/chromium/issues/detail?id=918340

deleted a duplicate fontdicts_mod.fini() call (#1538)

Add an initial .clang-format config

There are things can be improved on clang-format side I guess before
the full assertion, but is the best we can get for now I guess.

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

Minor, hack .editorconfig for vscode use

As VSCode and Atom https://github.com/Microsoft/vscode/issues/44438 don't support it

fix leak in subset_enc_supp_codes (#1537)

oss-fuzz issue 12310

[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