[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

[pragma] Enable error -Wdelete-non-virtual-dtor

Currently fails directwrite backend.

[directwrite] Fix error

hb-directwrite.cc: In constructor 'TextAnalysis::TextAnalysis(const wchar_t*, uint32_t, const wchar_t*, DWRITE_READING_DIRECTION)':
hb-directwrite.cc:489:18: error: 'TextAnalysis::mText' will be initialized after [-Werror=reorder]
   const wchar_t* mText;
                  ^~~~~
hb-directwrite.cc:488:12: error:   'uint32_t TextAnalysis::mTextLength' [-Werror=reorder]
   uint32_t mTextLength;
            ^~~~~~~~~~~
hb-directwrite.cc:282:3: error:   when initialized here [-Werror=reorder]
   TextAnalysis (const wchar_t* text, uint32_t textLength,
   ^~~~~~~~~~~~
hb-directwrite.cc: In function 'hb_bool_t _hb_directwrite_shape_full(hb_shape_plan_t*, hb_font_t*, hb_buffer_t*, const hb_feature_t*, unsigned int, float)':

[pragma] More cast-align whitelist

[pragma] Allow cast-align in hb-font.cc

../src/hb-machinery.hh: In instantiation of 'const Type& StructAtOffset(const void*, unsigned int) [with Type = unsigned int]':
../src/hb-font.cc:144:85:   required from here
../src/hb-machinery.hh:63:12: error: cast from 'const char*' to 'const unsigned int*' increases required alignment of target type [-Werror=cast-align]
 { return * reinterpret_cast<const Type*> ((const char *) P + offset); }
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/hb-machinery.hh: In instantiation of 'Type& StructAtOffset(void*, unsigned int) [with Type = unsigned int]':
../src/hb-font.cc:145:79:   required from here
../src/hb-machinery.hh:66:12: error: cast from 'char*' to 'unsigned int*' increases required alignment of target type [-Werror=cast-align]
 { return * reinterpret_cast<Type*> ((char *) P + offset); }
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/hb-machinery.hh: In instantiation of 'Type& StructAtOffset(void*, unsigned int) [with Type = int]':

[pragma] GCC diagnostic error   "-Wreorder"

[pragma] GCC diagnostic warning "-Wmaybe-uninitialized"

[pragma] Massage more

[pragma] More

[pragma] Reorg again

https://github.com/harfbuzz/harfbuzz/issues/1546

[pragma] GCC diagnostic ignored "-Wunknown-warning-option"

Try at fixing https://github.com/harfbuzz/harfbuzz/issues/1546

[uniscribe] More warnings

[uniscribe] Fix warnings

Fix build

[pragma] GCC diagnostic ignored "-Wclass-memaccess"

[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