[unsafe-to-break] Be careful with flag propagation when merging clusters

Set mask to 0, instead of 1, by default

This shouldn't matter.

[unsafe-to-break] Mark Indic-like clusters as unsafe-to-break

[unsafe-to-break] Allocate public bits from 0

[safe-to-break] Fix logic when there is no out-buffer

[unsafe-to-break] Simplify logic

Always use the algorithm for non-monotone case.  It's more robust.

[unsafe-to-break] Add UNSAFE_TO_BREAK flag

Not all shapers code is updated to set this properly.
GSUB and Arabic shaper are updated.
GPOS and other shapers are NOT.

Fixes https://github.com/behdad/harfbuzz/issues/224

[util] Fix two other Coverity warnings

[util] Fix warning

[ft] Fix theoretical leak

Treat HAMZA ABOVE similar to SHADD for sorting purposes

Part of https://github.com/behdad/harfbuzz/issues/509

Protect against div-by-zero in CBDT extent code

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=1452#c5

CC https://github.com/behdad/harfbuzz/issues/139

Add tests for 'avar' fix 5dc30451b80f6bb0079424a130875c10486e4d34

1.4.8

Two fixes to avar mapping

1. Handle segment with two entries correctly,

2. Fix rounding math.  Ouch!

Fixes https://github.com/behdad/harfbuzz/issues/521

[check-defs/symbols.sh] Drop empty-symbol lines

Fixes https://github.com/behdad/harfbuzz/issues/510

[hb-shape] Rename --show-messages to --trace

https://github.com/behdad/harfbuzz/issues/506

[hb-shape] Improve shaping-debug output

Before, that was printed using --debug (and in both hb-shape and hb-view).
Changed it, now hb-shape has a new command-line argument called --show-messages.
When invoked, it also respects other output formatting options.  The messages
are better formatted and printed to te same place that hb-shape output is
directed to.  Previously they were written to stderr.

Fixes https://github.com/behdad/harfbuzz/issues/506

Unbreak arm-none-eabi build again (#514)

1.4.7

[arabic] Adjust feature order again

Fixes https://github.com/behdad/harfbuzz/issues/505

[coretext] Allow to disable kern (#508)

* Minor

* [coretext] Fix leak

* [coretext] Do not reset num_features

* [coretext] allow to disable kern; re-enabling doesn't seem to be working

[use] Allow up to two medial-below letters

Fixes https://github.com/behdad/harfbuzz/issues/376

[use] Fix shaping of U+AA29 CHAM VOWEL SIGN AA

Part of https://github.com/behdad/harfbuzz/issues/376
Also see https://github.com/roozbehp/unicode-data/issues/6

Test added, using NotoSansCham built from Noto Phase III sources.

[indic] Add github URL

[indic] Fix https://github.com/behdad/harfbuzz/issues/478

[indic] Disable automatic ZWNJ handling for Indic features

Fixes https://github.com/behdad/harfbuzz/issues/294

Also fixes a bunch of other Indic issues.  Test results after:

BENGALI: 353725 out of 354188 tests passed. 463 failed (0.130722%)
DEVANAGARI: 707307 out of 707394 tests passed. 87 failed (0.0122987%)
GUJARATI: 366355 out of 366457 tests passed. 102 failed (0.0278341%)
GURMUKHI: 60732 out of 60747 tests passed. 15 failed (0.0246926%)
KANNADA: 951201 out of 951913 tests passed. 712 failed (0.0747968%)
KHMER: 299071 out of 299124 tests passed. 53 failed (0.0177184%)
MALAYALAM: 1048136 out of 1048334 tests passed. 198 failed (0.0188871%)
ORIYA: 42320 out of 42329 tests passed. 9 failed (0.021262%)
SINHALA: 271662 out of 271847 tests passed. 185 failed (0.068053%)
TAMIL: 1091754 out of 1091754 tests passed. 0 failed (0%)
TELUGU: 970555 out of 970573 tests passed. 18 failed (0.00185457%)

Before:

BENGALI: 353725 out of 354188 tests passed. 463 failed (0.130722%)
DEVANAGARI: 707307 out of 707394 tests passed. 87 failed (0.0122987%)
GUJARATI: 366349 out of 366457 tests passed. 108 failed (0.0294714%)
GURMUKHI: 60732 out of 60747 tests passed. 15 failed (0.0246926%)
KANNADA: 951190 out of 951913 tests passed. 723 failed (0.0759523%)
KHMER: 299070 out of 299124 tests passed. 54 failed (0.0180527%)
MALAYALAM: 1048136 out of 1048334 tests passed. 198 failed (0.0188871%)
ORIYA: 42320 out of 42329 tests passed. 9 failed (0.021262%)
SINHALA: 271662 out of 271847 tests passed. 185 failed (0.068053%)
TAMIL: 1091753 out of 1091754 tests passed. 1 failed (9.15957e-05%)
TELUGU: 970555 out of 970573 tests passed. 18 failed (0.00185457%)

[indic] Add infrastructure to disable ZWNJ-skipping in context-matching

Not used yet.

Shaping tests for Tibetan vowels (#446)

* Shaping tests for Tibetan vowels

* Test-cases for the Dzongkha contractions with multiple vowel-signs added.

* going to be removed

* Extended contraction-test-cases to all test cases in contractions.txt that actually use multiple-vowels (113 cases)

Fix for reordering of Tibetan vowel u (#443)

* Undone change for Tibetan vowel u

* removed comment on reordering that became invalid with roll-back

* Support for Dzongkha contractions with multiple vowel-signs

* Removed non-functional and unnecessary defines for HB_MODIFIED_COMBINING_CLASS_CCC138,140

Win32/NMake builds: Support builds from GIT (#498)

Add Python scripts to generate the full win32/config.h.win32 and
src/hb-version.h which can be used to build directly from a GIT
checkout.  Since the scripts are currently intended for building from a
GIT checkout, these are not distributed in the release tarballs.

Also, support the re-build of Ragel-generated .hh headers using the NMake
build system, and allow one to specify the path of the Ragel executable
if a suitable one cannot be found in the PATH.

Update the Win32/NMake build documentation to let people know about how
these mechanisms can be utilized.

Fix arm-none-eabi build (fixes #451) (#496)

Support branch prediction helpers on clang compiles (#491)

Handle allocation failure in hb-language code

Avoid using strdup inside library. (#488)

If an application provides a malloc replacement through
hb_malloc_impl() it is important that it is used to allocate
everything, but the use of strdup() circumvents this and
causes system malloc() to be called instead. This pairs
badly with the custom hb_free_impl() being called later.

Do not skip TAG characters in glyph substitution (#487)

Hide them like Mongolian Free Variation Selectors instead.

Fixes https://github.com/behdad/harfbuzz/issues/463

Minor

[cmake] Add framework build support (#484)

[cmake] minor (#482)

1.4.6

Fix graphite2 rtl conversion (#475)

[cmake] Final touches (#473)

Fix clang -Wcomma warnings (#471) (#472)

clang's new -Wcomma compiler option warns about possible misuse of the
comma operator such as between two statements.

hb-common.cc:190:9 [-Wcomma] possible misuse of comma operator here
hb-ot-layout-gsubgpos-private.hh:345:30 [-Wcomma] possible misuse of
comma operator here
hb-shape-plan.cc:438:26 [-Wcomma] possible misuse of comma operator here

[cmake] Remove HB_DISABLE_DEPRECATED as it seems needed for pango build (#470)

Use absolute paths of ragel generated headers (#467)

https://github.com/behdad/harfbuzz/issues/455

Small doc fix: `make check` runs the tests (#469)

[cmake] Fix try compile link issues (#466)

[cmake] typo (#465)

[cmake] Add atomic ops availability detection (#464)

[cmake] Remove NO_MT flag (#462)

Blacklist GDEF table in additional Tahoma versions. (#459)

There are more broken versions of Tahoma out there on various Windows releases,
so we need to add them to our blacklist to avoid broken rendering.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1279925 for details.

[cmake] Improve third party libraries support (#461)

[cmake] Add utils build support (#460)

NMake Makefiles: Fix ICU builds

Fix the check conditions in config-msvc.mak and info-msvc.mak so that
the ICU items does indeed get built into the HarfBuzz main DLL, and that
the correct configuration info is displayed.

Also update the checks in detectenv-msvc.mak so that we can detect that
we are using Visual Studio 2017 (although the 2015-built binaries use
the same CRT DLL as the 2017 ones).

Move list of ragel sources to Makefile.sources as well

Split ragel generated files lists and remove hardcoded rl files lists (#453)

Add CMake build support (#444)

avoid UBSan warning in get_stage_lookups (#450)

See https://bugzilla.mozilla.org/show_bug.cgi?id=1336600

Current fonttools (3.9.1) generate subset-file called font.subset.ttf instead of older font.ttf.subset

Updated samples: record-it.sh is now record-test.sh

[tools] Make hb-unicode-code work with Python 3

Related to https://github.com/behdad/harfbuzz/pull/445

Blacklist more versions of Padauk

Patch from Phil Race.

[ci] Fix msys2 build on AppVeyor

For whatever reason the env variables need to be the Windows way or they
end up being empty.

Just always use strtod here

src/hb-common.cc: Fix build on older Visual Studio

Visual Studio only supported strtof() from Visual Studio 2013 onwards, so
use strtod() instead to do the operation, which should do the same thing,
sans going to a double, not a float.

Visual Studio builds: Fix Introspection when UCDN enabled

The sources in src/hb-ucdn and not included correctly into the NMake
Makefiles, as we need their explicit relative location as we pass all the
sources we used into the introspection scanner.  This was not an issue
before as we excluded the UCDN sources in the build when we enabled
introspection (meaning GLib is enabled), but since we default on using
UCDN on all builds unless explicitly disabled, we need to deal with this.

This did not affect builds using UCDN without introspection due to the use
of NMake batch rules.

Fix this by creating a NMake Makefile module on-the-fly with the correct
subdir info, and using that list in there instead.

Update Visual Studio builds for UCDN usage

We now use UCDN by default, so make it so in the build files; however
don't hardcode HAVE_UCDN as one may still opt not to use it (but pass
it in as a CFLAG unless one explicitly disables UCDN by using NO_UCDN=1
on the NMake command line).

Note that we are not blocking builds where UCDN is disabled along with
GLib and ICU, as that will trigger a build error anyways which will tell
the user what needs to be done to remedy this.

1.4.5

Revert "Fix Context lookup application when moving back after a glyph delete"

This reverts commit b9b005f3a44ccf78a45b212b126287b69b9f4b40.

This introduced invalid access cases. Revert until I fix correctly.

Fix buffer serialize of empty buffer

Ouch!

1.4.4

Add TODO item

Remove TODO item that is not going to happen

Fix Context lookup application when moving back after a glyph delete

This was broken forever, since days that we did not allow moving
tape backwards. Works now. Reported by Doug Felt.

Add few tests found by libFuzzer and oss-fuzz

Fix buffer-overrun with Bengali reph positioning code

This has no security implications whatsoever since we always keep
and extra element at the end of buffer, just in case.

Discovered by oss-fuzz
CC https://github.com/behdad/harfbuzz/issues/139
Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=660

1.4.3

Blacklist another instance of Padauk (#419)

In https://crbug.com/681813 another instance of Padauk was identified
triggering collapsed glyphs. Blacklist this version by patching
hb-ot-layout.cc to print out gdef, gsub, and gpos table length, then
adding those to the list of blacklisted versions.

[ci] Disable vcpkg freetype installation and fix Appveyor CI (#422)

Guard against underflow when adjusting length (#421)

* Guard against underflow when adjusting length

With the fuzz-testcase in mozilla bug 1295299, we end up with a recursed lookup that removes 3 items, when `match_positions[idx]` is 0, which results in (unsigned) `end` wrapping to a huge value.

Making `end` a signed int is probably the simplest route to a fix.

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1295299.

* Add testcase for #421.

[indic] Add support for Grantha marks that may be used in Tamil to th… (#401)

* [indic] Add support for Grantha marks that may be used in Tamil to the Indic table.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1331339.

Testcase: U+0BA4,U+0BC6,U+1133c,U+0BAA,U+1133c,U+0BC6,U+1133c

* [indic] Add test for Grantha nukta that is allowed in Tamil by ScriptExtensions.txt

Unbreak hb-coretext build

Merge pull request #357 from khaledhosny/graphite-scale

[graphite] Fix shaping with varying font sizes

Set LC_ALL instead of LANG when creating harfbuzz.def

Hopefully fixes Travis faliure?
https://github.com/behdad/harfbuzz/pull/413#issuecomment-278481882
https://travis-ci.org/khaledhosny/harfbuzz/jobs/199785332#L1065

hb-font: Fix a potentially undefined use of memcmp() (#413)

While it’s fine to call memcmp(x, 0, 0) in practice, the C99 standard
explicitly says that this is not allowed: even if the length is zero,
the pointer arguments must be valid.

http://stackoverflow.com/a/16363034

Coverity ID: 141178

Signed-off-by: Philip Withnall <withnall@endlessm.com>

Update ax_pthread.m4 to latest upstream version

Might help with https://github.com/behdad/harfbuzz/issues/399

test: Fix some memory leaks in test-font.c (#409)

Coverity IDs: 141039, 141040, 141041

util: Add missing field initialisers in constructor (#410)

Coverity ID: 141042

Silence Coverity warning

Fixes https://github.com/behdad/harfbuzz/issues/408

Shuffle things around a bit

Minor

Add dirty-state tracking to hb_face_t

Add dirty-state tracking to hb_font_t

Add new API hb_font_set_face()

For completeness.

Add Win10 Anniversary Update version of Tahoma to GDEF blacklist. (#412)

Minor

Fixes https://github.com/behdad/harfbuzz/issues/411

Route Adlam through Arabic shaper

Fixes joined Adlam rendering.

Fixes https://github.com/googlei18n/noto-fonts/issues/828

Add Win7 version of himalaya.ttf to the GDEF table blacklist. (#407)

[var] Implement MVAR table

Unhooked. Untested.

[var] Whitespace