Merge branch 'master' into iter

Minor, use a meaningful naming in template parameter (#1582)

Fix djgpp complains by tweaking templates (#1579)

For some reasons djgpp doesn't understand "unsigned int" can be same
with one of uint*_t anyway so lets do that for it explicitly.

Just to note, our CI's djgpp is based GCC 7.2.0 and isn't old.

hb-coretext.cc: remove TARGET_OS_MAC from test (#1578)

it is always true when building on APPLE systems
and this file only builds on APPLE systems

[ci] Install 'base-devel' on ArchLinux CI bot

[ci] Install 'which' in ArchLinux bot

[ci] Speculative fix for Alpine and ArchLinux bots

[ci] Disable the just added bot

Doesn't play well with CircleCI apparently

https://circleci.com/gh/harfbuzz/harfbuzz/74289

[ci] Add a Void Linux bot

https://voidlinux.org/ is yet another and different distro written from scratch, thus completely eligible to have a bot here!

Seriously however not that useful yet however but I will try to overload it other tasks later, like our other bots.

No test yet also, couldn't install cairo-devel, will try to fix that later.

Their harfbuzz package source: https://github.com/void-linux/void-packages/blob/master/srcpkgs/harfbuzz/template

Merge pull request #1571 from kencu/cleanostests

hb-coretext.cc: clean up macosx test

hb-coretext.cc: clean up macosx test

TARGET_OS_OSX was introduced only in late OS versions
so always returns as "0" on older systems.

if !TARGET_OS_IPHONE can work, as it returns as !0 on older
systems where TARGET_OS_IPHONE is not defined, but is not
specific

if TARGET_OS_MAC && !(defined(TARGET_OS_IPHONE) && TARGET_OS_IPHONE)
is both specific and accurate on all systems.

Adjust hb_is_signed<>

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

Remove stale comment

[iter] Fix mystery crash

Fuzzer caught it:

==14==ERROR: AddressSanitizer: stack-use-after-return on address 0x7fca2ed7a3e0 at pc 0x0000006057aa bp 0x7ffc3290f1d0 sp 0x7ffc3290f1c8
READ of size 4 at 0x7fca2ed7a3e0 thread T0
SCARINESS: 55 (4-byte-read-stack-use-after-return)
    #0 0x6057a9 in OT::SingleSubstFormat2::subset(hb_subset_context_t*) const /src/harfbuzz/src/./hb-ot-layout-gsub-table.hh:194:40
    #1 0x5ff921 in hb_subset_context_t::return_t OT::SingleSubst::dispatch<hb_subset_context_t>(hb_subset_context_t*) const /src/harfbuzz/src/./hb-ot-layout-gsub-table.hh:256:13

I can't reproduce locally, but many of the bots are failing because of this
as well.

It's a pity that operator-> must return pointer.  Ugh.  Why?!

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.

Merge branch 'master' into iter

[configure] Fix up

[configure] Fix up

Merge branch 'master' into iter

[configure] Print compiler version info in report

[configure] Print compiler version info in report

[ci] Switch clang-O3-O0 bot to Ubuntu image

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

[iter] Warning fix

Not sure why I don't get it, but this warning:

warning: base class ‘struct hb_iter_fallback_mixin_t<hb_array_t<const OT::UVSMapping>, const OT::UVSMapping&>’ should be explicitly initialized in the copy constructor [-Wextra]

[ci] Disable -Wunused-template on -Weverything bot

./hb-algs.hh:37:3: error: unused function template 'operator()' [-Werror,-Wunused-template]
  operator () (const T& v) const { return v; }
  ^

Merge pull request #1564 from googlefonts/retain_gids

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

2.3.1

[iter] Whitespace

[iter] Add hb_iter_with_fallback_t instead

[iter] Add hb_iter_with_mixin_t<>

[iter] Export operator << / >>

[iter] Rename

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

Move hb_addressof() to hb-meta.hh

Merge branch 'master' into iter

[meta] Remove _ft struct names

Using decltype() instead.

[pair] Use decltype

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

[iter] Add operator << to set / vector

[iter] Fix operator() impls

[iter] Add operator>> and operator<<

[iter] Use auto c = C.iter() internally

Merge branch 'master' into iter

[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

Try fixing MSVC build

[iter] Accept C arrays in hb_iter()

[test] Test unary operator+

"Test" as in compiles..

[iter] Fix hb_iter()

[iter] Add unary operator+ that returns a copy

[iter] Add hb_bool() and make hb_filter default to it for predicate

[meta] Back to using _ft suffix for function-object types

Seprate namespace, cleaner, more clear.

[iter] Test that default-constructed iterators are empty

pragma GCC diagnostic error   "-Winjected-class-name"

See 6b6783e1588ebe5772a1edc19552219e9d931bda

[iter] Use hb_declval() instead of Null() to get instance

I had used Null to make one of the bots happy before.  Not going
to bend to such demands anymore..

[iter/meta] Fix build on newer clang

The mystery failure had to do with SFINAE failure because the template
function involved was accessing ::iter_t of a type that was also named iter_t.
In this context, apparently:

warning: ISO C++ specifies that qualified reference to 'iter_t' is a
constructor name rather than a type in this context, despite preceding 'typename' keyword
[-Winjected-class-name]

We use a new macro, also called hb_iter_t(), to get iterator type of
a type.  This uses declval/hb_decltype, and has the added benefit
that it returns correct type for const vs non-const objects, if they
have different iterators.

[meta] Mark function-objects as const

Merge branch 'master' into iter

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

Fix mac test

More static constexpr

Merge branch 'master' into iter

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

[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

[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