[subset] Merge remote-tracking branch 'googlefonts/master'

[subset] Don't fail on different checksum adjustment in subsetting tests.

[subset] Fix hhea subsetting and clean up some memory leaks.

[subset] Add missing face reference in hb-subset-plan plus ensure all struct members are cleaned up on destroy.

[subset] In hb-subset-test use hb_set_union instead of manually copying set.

[subset] Restore hb_face_data_destroy to be internal.

[subset] Fix memory leak in hb-ot-{maxp,os2}. Plus some formatting.

Another linker tweaking

Trying to fix this warning:
cc1: warning: command line option ‘-fno-threadsafe-statics’ is valid for C++/ObjC++ but not for C

Let's see what the clang bots think of it.

[set] Tune size again

Switch from a 128+64 hierarchy, to 32+32. New page size is 1024.
This should make set iteration faster, without affecting other op
performances much.

[subset] remove TODO that was already done

[subset] maxp wrong int type, note to use copy_writable_or_fail

[subset] return bool not hb_bool_t from table::subset

[subset] Use a supplier instead of memcpy and fix a few unnecessary {}s for cmap

[subset] add free

[subset] Give Behdad credit again

[subset] First pass at hmtx

Redirect email notifications to another mailing list dedicated for this (#790)

Add .collect_logs and use it on CI bots, fixes #786 (#789)

[subset] Fix codepoint iteration in hb-subset-test.

Another try

Fix compile error in hb-subset-test.

[test] Fix warning

Tweak Makefile some more

[subset] Allow linking to libstdc++

We want to use it.

copy all cmap groups at once

Merge remote-tracking branch 'upstream/master' into fixed

[subset] Drop GSUB, GDEF, GPOS, and DSIG from subsetter expected test outputs.

[subset] capitalize dsig.

[subset] Extract maxp subsetting into hb-ot-maxp-table.

[subset] Add a test for OS/2 fixes during subsetting.

[subset] white and add inline in hb-ot-cmap-table.

[subset] Correct usFirstCharIndex and usLastCharIndex in OS2 table when subsetting.

Another attempt on fixing the tests on Travis (#788)

[set] Add backwards iterator

New API:
- hb_set_previous()
- hb_set_previous_range()

[set] Allow starting iteration from a non-member of the set

Add hb_blob_copy_writable_or_fail()

New API:
- hb_blob_copy_writable_or_fail()

[subset] Rename constructor to hb_subset_input_create_or_fail()

To signify that unlike rest of library, returns nullptr on failure.

[use] Add dump-use-data

[khmer] Add dump-khmer-data

[myanmar] Add dump-myanmar-data

[indic] Rename print-indic-table to dump-indic-data

Makes more sense given what this prints, err, dumps.

[indic] Explicitly number indic_position_t

So it's easier to cross-reference from output of print-indic-table.

More tweaking linker and linker flags

[indic] Share some copy/pasta code

[indic] Add print-indic-table

[subset] Fix loca offsets

Was writing source font's offsets. Ouch.

Another tweak at libstdc++ things

To fix this warning:

cc1: warning: command line option ‘-fno-threadsafe-statics’ is valid for C++/ObjC++ but not for C

[aat] Minor

Minor

[subset] hb_subset_input_t changes

Disable threadsafe static's for all libraries

Even in libharfbuzz-subset, we still will handle our own threadsafety
for statics.

Specify the project name for coverity, trying to fix #785

Pass no-libstdc++ flags only to libraries suppressing it

[subset] Allocate bulk memory with malloc(), not calloc()

We use calloc() to allocate simple structs and set them to zero.
For bulk memory, use malloc() that doesn't zero.

Use hb_face_get_upem() instead of face->get_upem()

Such that hmtx accelerator can be used from libharfbuzz-subset.

[subset] Remove unreachable code

sanitizer.sanitize never return nullptr.

Allow libharfbuzz-subset to link to libstdc++

We want to use it instead of reinventing hashmap...

Also, don't change linker for ICU since libicu also uses libstdc++.

Move linker specifying to a better place

[subset] whitespace

output format 12 as enc 10 to match how Roboto did it

basic wiring for a (failing) cmap test

auto-completed the wrong gids_to_retain

Use functions to get new gids. Avoid 0; fonttools drops it from cmap.

[subset] Remove test-subset, testing is planned to be done at the table level here with test/subset covering the complete subsetting operation.

[subset] added todo in test-subset-glyf.

[subset] re-enable test-subset-glyf. Refactor to use common functions in hb-subset-test.h

[subset] Add some helper functions for writing tests around subsetting.

[subset] Fix loca generation, was previously writing the original glyph starting offset.

Rename --with-stdcpp → --with-libstdc++

Also improve wording a bit and fix make check in --with-libstdc++=yes.

we love all our groups

drop GDEF, GPOS, GSUB, and dsig

[subset] Set the new number of glyphs in maxp.

Don't force loca version to long.

Destroy the subset plan at the end of subsetting.

Build a working cmap format 12

subset for real

C-style comments

Fix symbol exporting issue on msys2 and ✅ bringing all the bots (#781)

Minor

Minor, don't skip the whole test when one .def file not found

Apply check-defs.sh to -icu and -subset, toward mingw fix (#780)

Add --with-stdcpp for ossfuzzer and mupdf use (#770)

minor

Add Travis based Coverity trigger (#778)

Remove linker selection from hb-ucdn

Since this is a convenience library, it's not actually linked. Or so I think.
Hoping this might fix this Appveyor failure:

https://ci.appveyor.com/project/harfbuzz/harfbuzz/build/job/9884jyb5vryja9pw

Making all in hb-ucdn
make[4]: Entering directory '/c/projects/harfbuzz/src/hb-ucdn'
  CC       libhb_ucdn_la-ucdn.lo
  GEN      libhb-ucdn.la
/bin/sh: libhb_ucdn_la-ucdn.lo: command not found
make[4]: *** [Makefile:404: libhb-ucdn.la] Error 127
make[4]: Leaving directory '/c/projects/harfbuzz/src/hb-ucdn'
make[3]: *** [Makefile:2548: all-recursive] Error 1
make[3]: Leaving directory '/c/projects/harfbuzz/src'
make[2]: *** [Makefile:1382: all] Error 2
make[2]: Leaving directory '/c/projects/harfbuzz/src'
make[1]: *** [Makefile:486: all-recursive] Error 1
make[1]: Leaving directory '/c/projects/harfbuzz'
make: *** [Makefile:418: all] Error 2

Add MSVC to no-visibility list

Add Sun compiler to no-visibility as well

Fixes https://github.com/harfbuzz/harfbuzz/pull/776

Fix more warnings

Another try at fixing multiple-defintion of NullPool

This one might do it...

[docs] Minor update

[subset] Another fixup

I broke this in c31fcf4c58d96eb7d9781a986991b1a79ac7be44

Try at fixing multiple-definition of NullPool

If we don't have visibility, make NullPool static.

[subset] Use auto_array_t where it does make sense

Better fix for https://github.com/harfbuzz/harfbuzz/pull/775

Minor

Install fonttools on two bots and enable cmake hb-subset test (#777)

Initialize bad_indices on hb-subset (#775)

fix gobject compile issue (#773)

[gpos] Adjust mark attachment heuristic

Hopefully it...
fixes https://github.com/harfbuzz/harfbuzz/issues/740

More fixes

Misc warning fixes