removed CMakeCache.txt

Silence Codacy

subset FDSelect and FDArray

Added a set of sources hb-subset-cff-common-private.cc & .hh for FDSelect subseting code.
Added FDSelect format 4 (CFF2 only) support. Shared its implementation with format 3 as a template.

silence gcc

Removed parsed hint fields from CFF2PrivateDictValues

Instead moved & stored uniformly in values vector (renamed from opstrs)
This should be fine as we aren't likely going to use them as hints after all
Templatized DictValues whether to parse them out
For subsetter, they aren't parsed

templatized CFF2 accelerator for lean Private dict interpretting for subset

also fixed local subrs serialization

removed TRACE_SERIALIZE from non-member function _write_cff2

Merge branch 'master' into cff-subset

more silencing fixes

silence archlinux

silence Codacy

Fix for recent rename

[uniscribe] Update for recent changes

Remove unused table reference

Minor

Make lazy-loader safe for nil objectification

Minor

Reduce storage by sharing face amongst lazy_loaders

Internal templatization of lazy-loaders

Remove OT namespace from hb-machinery-private.hh

Minor

[face] Sprinkle const in the API

Rename

Add comment re (our only) race condition

removed unused field OpStr::update

fixed Index::serialize

The bug was causing CharString of the last glyph corrupt

fixed CFF2VariationStore::serialize

also get_size

1.8.5

[atomic] Fix Solaris ones to add proper barriers

[atomic] Remove volatile from IBM impl signature

[atomic] Oops

[atomic] Use read-barrier for get()

[atomic] Some more minor tweaks

[ci] Fix Appveyor bot (#1123)

Implemented hb_subset_cff2

Added serialize functions to CFF2 structs
Fixed issues & bugs & code cleanup
Now subsetting CFF2 table outputs something

Revert "Add a new API, hb_ot_layout_get_feature_name_ids (#976)" (#1121)

This reverts commit 0c1b287b72e91e0898d75acb5d5acf1c6b9a7498.

[atomic] Fix cast to fallback ptr_get()

[gobject] Fix copy/paste error

[atomic] On IBM, use light-weight sync for everything

lwsync() is a full read/write-barrier.  That's all we need, never
need sync().  I'm not sure why an isync() was used in fetch_and_add,
but since that's a read-modify-write, I just changed it to have
lwsync() on both sides.

[atomic] Add XXX items around Solaris ops

Since add_int and cas are both read-modify-write, I wonder if we
also need a barrier after them.

[atomic] Fix get() impl

Originally, glib's atomic_get was implemented as "memory_barrier; load".
I copied this into cairo, fontconfig, and harfbuzz.  However, that's
wrong.  Correct way is "load; memory_barrier".  The details are long
and hard to fully grasp.  Best to read:

  https://www.kernel.org/doc/Documentation/memory-barriers.txt

Also see my report against GNOME:

  https://gitlab.gnome.org/GNOME/glib/issues/1449

Note that this is irrelevant if C++11-like atomic ops are available.

[gobject] Hook up hb_map_t

[atomic] Fix fallback impl

[atomic] Add get_relaxed / set_relaxed

To help TSan and be more "correct".

[atomic] Use CONSUME, not ACQUIRE, memory-order for get()

Although, all implementations just elevate that to ACQUIRE.
But requirement for us is just CONSUME.

Revert "[atomic] Make pointer get op relaxed instead of acquire"

This reverts commit b1e5650c67266dc158f22355fed206cd1c413f70.

After lots of head-scratching and finally finding the only truly
readable source to be the good old:

  https://www.kernel.org/doc/Documentation/memory-barriers.txt

I've convinced myself that we need consume memory-ordering on get().
The location of memory-barrier in a load should be after, not before
the load.  That needs fixing.  I'll do that separately.

[subset] limit the max codepoint value to the unicode limit.
When collecting all codepoints in the cmap avoid using large amount of memory for fonts that declare coverage over all 32 bit integers.

[subset] Add hb_ot_layout_collect_features to harfbuzz-sections.txt.
Add the fuzzer test case for feature collection timeout.

[subset] use add_array to populate feature_indexes.
This is much faster then calling a bunch of individual add()'s.

[subset] collect features first, then use those to collect lookups.

[indic] Flip default logic for double-halants in old-school

Oriya went down from 9 to 2.

BENGALI: 353725 out of 354188 tests passed. 463 failed (0.130722%)
DEVANAGARI: 707311 out of 707394 tests passed. 83 failed (0.0117332%)
GUJARATI: 366355 out of 366457 tests passed. 102 failed (0.0278341%)
GURMUKHI: 60729 out of 60747 tests passed. 18 failed (0.0296311%)
KANNADA: 951300 out of 951913 tests passed. 613 failed (0.0643966%)
MALAYALAM: 1048136 out of 1048334 tests passed. 198 failed (0.0188871%)
MYANMAR: 1115830 out of 1123883 tests passed. 8053 failed (0.716534%)
ORIYA: 42327 out of 42329 tests passed. 2 failed (0.00472489%)
SINHALA: 271596 out of 271847 tests passed. 251 failed (0.0923313%)
TAMIL: 1091754 out of 1091754 tests passed. 0 failed (0%)
TELUGU: 970555 out of 970573 tests passed. 18 failed (0.00185457%)

[indic] Allow double-halant in old-spec Devanagari

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

[test] Add Khmer tests, with NotoSansKhmer-Regular.ttf

Note that there's minor positioning differences, and ONE reordering
difference between what we get for these and what Uniscribe gets.
Probably same as what's described in commit message for
1a96cc825dc9c8e3b6eef1403fe0864a1cfc0245

[khmer] Typo

[test] Add Khmer test texts from recent bugs

[test] Move things around for shaper updates

[khmer] Clear syllables before presentation features

Probably not what Uniscribe does, but good idea?

[khmer] Revert previous change, and remove pauses

This makes test suite happy again (at 44) while fixing the sequences
we were fixing, which were the following with KhmerUI.ttf:

  U+1789,U+17BC
  U+1789,U+17D2,U+1789
  U+1789,U+17D2,U+1789,U+17BC

Fixes rest of https://github.com/harfbuzz/harfbuzz/issues/974

[khmer] Apply ccmp after basic features

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

[khmer] Rewrite most of shaper to better follow spec

Khmer spec has only one reordering phase, and only simple prebase
matra and Coeng-Ro reordering.  Implement that.  Specifically,
this was done to address recognizing different orders of the matra
and Coeng-Ro sequence.  That said, some combinations are now
reordered differently from Uniscribe.  Not clear if that's intended
or a bug in Uniscribe.  The following two sequences render the same
in Uniscribe whereas we reorder them differently:

  U+17A0,U+17D2,U+179A,U+17C2
  U+17A0,U+17C2,U+17D2,U+179A

For that reason, our test suite numbers regressed slightly.  Used
to be at 34 for fails, now at:

KHMER: 299080 out of 299124 tests passed. 44 failed (0.0147096%)

But generally a good change, and removed lots of code.

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

[shaper] Move code around

[shaper] Remove a macro

[atomic] Make pointer get op relaxed instead of acquire

We only use it before cmpexch, so relaxed is fine and faster for
common case.

[serialize] Increase stage count from 8 to 32

Indic shaper uses many stages.  Now we are provably not limiting
functionality whereas the previous limit of 8 was assuming real-world
practices.

[serialize] Remove unused truncate() method

fixed FDSelect0::sanitize

Added sanitize functions & calls

Added sanitize functions to FDSelect
Added satnitize calls for FDSelect, VariationStore, FDArray, TopDict

Merge branch 'master' into cff-subset

[subset] During lookup collection remember the features we've already processed.

Minor

[serialize] Allocate for markFilteringSet

Minor

Fix unlikely leaks

Add hb-iter-private.hh

Unused so far.

Move some code from hb-open-type-private.hh to hb-machinery-private.hh

[closure] Allocate out set on the stack

Fix compile

[coretext] Add note

[subset] Only used reachable lookups for gsub closure.

[closure] Separate in and out glyphs

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

[subset] Limit the iterations of the closure algorithm.
Prevents O(n^2) run times.

Fix visibility on mingw32

Should fix bots again.

Fix infinite loop in loading num_glyphs within sanitizer

One more visibility trick

Should fix Windows build again.

One more

Remove unnecessary OT:: namespace specifiers

Set num glyphs on sanitizer reference_table()

Move out-of-class definitions of two methods to hb-static so they
are accessible in libharfbuzz-subset.

Remove Sanitizer<>

Port rest of code away from Sanitizer<>

Port dump-emoji away from Sanitizer<>

.

Minor

Port sbix off of Sanitizer<>

Port more off of Sanitizer<>

Add sanitizer reference_table, also fix clang build

Clang is being really peculiar.  Fix with:

-  inline hb_blob_t *sanitize (hb_blob_t *blob) { return c->sanitize<Type> (blob); }
+  inline hb_blob_t *sanitize (hb_blob_t *blob) { return c[0].template/*clang idiosyncrasy*/sanitize_blob<Type> (blob); }

Order Chakma split vowels in accordance with NFC

Fixes #1105.

Add a new API, hb_ot_layout_get_feature_name_ids (#976)

This new API returns cvXX and ssXX related NameId, things like
featUiLabelNameId, featUiTooltipTextNameId, sampleTextNameId, ... of cvXX
and UINameId of ssXX, in a unified way.

However HarfBuzz currently doesn't expose an API for retrieving the actual
information associated with NameId from the `name` table and that should be
done separately.

first push of CFF/CFF2 work

Index, Dict structs
hooked up to hb-subset (takes CFF2, outputs empty CFF2)

Remove dump-fon (#1100)

It had interesting stuffs like EXE parsing and
big-endian parsers but added in an attempt to find
a solution for #681 which later found not related.

Remove AAT's gcid/fmtx (#1099)

We are not going to use them, at least in the foreseeable future