[subset] Minor

Merge pull request #754 from googlefonts/master

Fix build for subsetting code.

ssize_t -> size_t

Switch to fopen() instead of open() in subset test. open wasn't compiling on windows.

Rename HB_SUBSET_sources -> HB_SUBSET_CLI_sources to match what cmake is looking for.

Wrap unistd.h include in ifdef.

Add libharfbuzz-subset.la to linking when building api tests.

[subset] Build hb-subset into separate libharfbuzz-subset.so

[subset] Minor

Merge pull request #745 from googlefonts/master

Interface for hb_subset, skeleton for the hb-subset cli, and basic testing rigging.

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

[aat] Fix unsafe-to-break

At any position, if state is not zero, mark unsafe-to-break before,
unless we can reason it safe.

At any position, if there's an action entry for end-of-text, mark
unsafe to break.

Also changes buffer diff impl to allow for flag differences as long
as the buffer glyph flags are superset of reference glyph flags.

With this, all MORX tests pass.

Actually call hb_subset

Add CMakeLists.txt to dist files for subset test.

Add CMake configuration for all of the new subsetting code.

Add missing unistd header to hb-subset.

Properly include subset test data files.

Re-write hb-subset utility to use main-font-text driver.

Whitespace

Include subset test files in distribution.

Whitespace

Get test-subset to pass.

Attach add a hb_face_t to hb_subset_face_t.

Whitespace

Add a basic implementation of hb-subset to enable compilation of test-subset.

Make the expected output for subsetting basics test be equal to the input file for now.

Add the beginning of a unit test for hb-subset

tweak test failure output. write to a temp file not stdout. test still fails because expected is not just an identical copy of input

Add hb-subset.h to hb.h

Fix typo in hb-subset.h

Remove basic subset test from XFAIL

Whitespace

Configure automake to run the new subset tests.

WIP test runner for subset tests.

In generate-expected-outputs read the test definition with utf8 encoding.

Add basic directory structure for subsetter integration tests. Plus a utility for generating expected output files.

Start to sketch APIs for subsetting

[aat] More adjustment to ContextualSubtable

See comment.

With this, MORX-20 passes if I turn --verify off.  Our unsafe-to-break
logic is currently broken in presence of end-of-text actions.  That's,
ugh, extra work to fix.  Let me try...

[aat] In ContextualSubstitute, apply end-of-text action to last glyph

[aat] Remove unneeded check

[configure] Fix wording

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

[aat] In ContextualSubtable, mark mark after substituting mark

Fixes MORX-21.

[aat] Fix ContextualSubtable sanitization

Fixes MORX-18, MORX-19, and MORX-22.

Correctly show documentation build status

Correctly show if building documentation is enabled or not in configure
summary.

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

Enable use of atexit() on macOS and related platforms.

The atexit() man page indicates that this is expected to behave
in the expected way on unloading of shared libraries.

Use nullptr, not NULL.

[cmake] unistd typo fix (#747)

Let VS1 follow U+1031 MYANMAR VOWEL SIGN E

Allow optional ZWJ in virama terminated cluster

Use nullptr instead of 0.

Fix typos.

[ot-layout] Fix nullptr dereference.

If the `calloc` for `gsub_accels` or `gpos_accels` fails, then the
unlikely branch afterwards can be taken, which frees up the
`hb_ot_layout_t`, but since those fields can now be `nullptr`, then
we don't want to dereference them.

[coretext] Fix memory leaks.

In `reference_table`, if the data is empty and we return early,
we still need to release the data object.

In `hb_coretext_shape`, there two edge cases where an early
return should release the attributed string.

1.7.5

Fix distcheck

[aat] Limit number of DontAdvance transitions

Remove the faulty loop-detection logic. Use max-ops for breaking
out of infinite loops or heavy work.

https://twitter.com/behdadesfahbod/status/958065026262183936

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

[cmake] Build src/ executables (#733)

[test] Use 'otool -L' where ldd doesn't exist (macOS) (#732)

Fix Travis build on macOS (#731)

Misc warning fixes

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

Use KernSubTableFormat2::rightClassTable

Issue originally reported at http://cppfiles.com/chromium.txt .

This fixes what appears to be a copy - paste issue which causes both
KernSubTableFormat2::rightClassTable and the 'right' argument to
KernSubTableFormat2::get_kerning to go unused.

[cmake] Use -fvisibility-inlines-hidden instead (#726)

[cmake] fix symbol exporting issue on newer gcc (#724)

[ci] minor, less hacks (#722)

[ci] minor (#721)

[ci] unbreak bots with outdated freetype version (#718)

Fix typos.

[aat] Fix Ligature matching

I hope... Makes "ffi" work with Zapfino. I'm not sure if doing it correctly though...

[aat] Add TODO

[aat] Implement LigatureSubtable

We form the Zapfino ligature now. Yay! No further testing done.

[aat] Embed a sanitizer context in hb_aat_apply_context_t

For run-time checking.

[aat] Add hb_aat_apply_context_t

Rename hb_apply_context_t to hb_ot_apply_context_t

Whitespace

[cmake] Always put test/api/CMakeLists.txt on distributions

hb-common.cc: Hatran script is right-to-left (#714)

"Hatran is written from right to left horizontally"

http://www.unicode.org/L2/L2012/12312-n4324-hatran.pdf

This ancient script was added with Unicode 8.0.

Also this spreadsheet (referenced in an inline comment in the
source code) has it as RTL: http://goo.gl/x9ilM

Improve avoiding C++ linkage, definition creation and cmake tests (#710)

Update record-test.sh to the new directory layout (#716)

[aat] Allocate set dynamically

Typo

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

[cmake] don't link to libstdc++ (#707)

[cmake] Fix for CMake 3.0 and below (#706)

[cmake] Fix symbols visibility and enable check-symbols.sh test (#705)

Disable MORX-13/14

Really, need better way to catch these :(.

Disallow sanitize recursing into Extension subtables multiple times

Fixes https://oss-fuzz.com/v2/testcase-detail/5216838347653120
which is a stack overflow, not by way of infinite recursion, just
being deep.  That's disallowed anyway, so catch it as it happens,
not afterwards.

Whitespace

[aat] Sanitize classTable in StateTable

[aat] Wire up table length to apply()

[aat] Towards implementing LigatureSubtable

Fix set initializer

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

[aat] Fix some struct sizes

I hate it that with my compiler at least, it doesn't catch totally uncompilable
statements in templates if they are not used...

[aat] Flush rest of buffer if state machine failed

Minor

[aat] Fix symbol visibility

Not sure why only one of the bots sees this as visible...

Revert accidental morx enablement

Fix docs

[aat] Tweak previous commit

[aat] Detect infinite-loops in state machine