Merge branch 'master' into cff-more-arrayof-fixes

Merge pull request #1567 from googlefonts/fuzzer

[subset] Update the subset fuzzer to get  options to use from test case.

[subset] Update the subset fuzzer to determine which options to use based on data in the fuzzing test case.
Add support for toggling retain_gids.

[configure] Fix up

[configure] Print compiler version info in report

Merge pull request #1564 from googlefonts/retain_gids

[subset] Add --retain-gids option to the subsetter.

2.3.1

[subset] Add expected files for retain-gids integration tests.

[subset] For retain gids don't truncate glyphs past the highest requested subset glyph.

[subset] Add retain-gids option to hb-subset executable.

[subset] Add some subsetting integration tests covering retain gids.

[subset] A few small fixes for the new subset plan api.

[subset] Update hb-subset-plan.cc to match hb-subset-plan.hh.

[subset] Update hb-subset-glyf.cc to use new hb-subset-plan API.

[subset] Update gsub to use glyphset() method of subset plan.

[subset] Correct maxp num glyph's to use new subset plan method.

[subset] Re-add glyphs array to subset plan, with new name 'glyphs_deprecated'. Switch CFF subsetting to use it.

Re-implement HMTX/VMTX subsetting. Update it to support glyph renumbering and simplify the implementation.

Update hb-ot-vorg-table and hb-ot-layout-common to use the updated subset plan api.

Remove glyph array from subset plan, make num_glyphs and glyphset private.

[subset] Fix hdmx subsetting when retain gids is enabled.

[subset] Compute num_glyphs during subset plan construction.
Update maxp to use the correct num glyphs.

[subset] Add unit test for glyf subsetting and retain gids.

[subset] fix failure to init instruction ranges values for an invalid glyph.

[subset] Do some refactoring in hb-subset-glyf.cc.
- Extract code out into helper methods in several places.
- Bundle loca address, size and is short into a struct.

Add retain_gids option to subset input. Update glyf and loca handling to respect retain_gids.

Merge branch 'master' into cff-more-arrayof-fixes

[CI] Peg CircleCI badge link to master branch

[skip ci]

[ci] Peg build status badges to master branch

[skip ci]

Switch README / README.md

[skip ci]

[uniscribe] Whitelist function type cast

Fix sign comparison error

Fix warning

c:\projects\harfbuzz\src\hb-ot-color-cbdt-table.hh(59): warning C4146: unary minus operator applied to unsigned type, result still unsigned [C:\projects\harfbuzz\build\harfbuzz.vcxproj]

[pragma] Silence MSVC unknown-pragma warning

Fix -Wcast-function-type warnings in util/ with gcc 4.8

Fix mac test

More static constexpr

[AAT] Add test for recent Ligature stack fix, using Zapfino on Mac

Merge branch 'master' into cff-more-arrayof-fixes

[AAT] Minor sign

[AAT] Use a ring buffer for ligature stack

I think Apple does very similarly, but probably with a stack size of 16.
We do it with a stack size that is currently set to 64.

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

Whitespace

[AAT] Handle transition errors during machine operation

Before we used to give up.  Now, just ignore error and continue processing.

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

Merge branch 'master' into cff-more-arrayof-fixes

[AAT] Ignore machine errors and continue

[AAT] Minor

[AAT] Handle out-of-bounds classes

[AAT] Minor

[AAT] Minor

More pragma control

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

Comment

Fix test

Form cluster for Emoji sub-region tag sequences

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

Fix macos tests with previous commit

Adjust mark offsets when zeroing from fallback mark positioning code

Adjust tests.

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

Merge branch 'master' into cff-more-arrayof-fixes

more rewriting with ArrayOf<>

CFF1 Encoding0, Encoding1, CFF1SuppEncData

[AAT] In InsertionChain, set mark to previous-position if inserting

Fixes MORX-31

[AAT] Re-enable tests MORX-31 and MORX-41

[AAT] Remove mark_set from InsertionChain

[test/text-rendering-tests] Update from upstream

More of previous

Fix sign-compare error resulted from promoting unsigned integers to signed larger ints

Clang and gcc know not to warn in these cases, but not nonmainstream compilers

Cast HB_UNTAG results to uint8_t

More -Wcast-error fix

Better fix for -Wcast-align errors

More -Wcast-error fixes

pragma GCC diagnostic error   "-Wsign-compare"

Fix cast-align error

If compiler doesn't inline StructAtOffset, this was an error since we
only disable cast-align at call-site.  So, move the cast out.

../src/hb-machinery.hh: In instantiation of 'const Type& StructAtOffset(const void*, unsigned int) [with Type = unsigned int]':
../src/hb-font.cc:146: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:147: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); }
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[CI] Remove gcc 4.2 bots

We don't support that version anymore.

Convert all other enum class consts to static constexpr

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

Convert unsigned enum class consts to static constexpr

Part of https://github.com/harfbuzz/harfbuzz/issues/1553

Convert tag enum class consts to static constexpr

Part of https://github.com/harfbuzz/harfbuzz/issues/1553

Convert boolean enum class consts to static constexpr

Part of https://github.com/harfbuzz/harfbuzz/issues/1553

Make some enum constants unsigned

Whitespace

pragma GCC diagnostic error   "-Wvla"

Fix grammar

Grammar fix

Remove wrongly added files

Use enum for class constant

[CFF] Use enum for tableTag

Use static constexpr for large class constants

Regular spaces will do

Adding leading space for code example

Fix description of example

Added option to build docs to docs

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