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

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

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

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

Fix hb_face_count() sanitize referencing

Remove TODO that is not gonna be done

Previously the idea was to cache sanitize results externally (think,
in Fontconfig) and avoid resanitizing every time.  That's, not a good
idea.

Move sanitizer code around a bit

[aat] Update for blob changes

Also, uncomment code again, just "if (0)" it out, so it doesn't get stale again.

1.8.4

Reland "Implement C++11-style GCC builtin atomic ops"

Fixed the crasher in it.

Minor

Revert "Implement C++11-style GCC builtin atomic ops"

This reverts commit 21fa170f0bfb0109c506ed17f5aff8b062564ffa.

Is crashing.  Oops.

Implement C++11-style GCC builtin atomic ops

Hopefully fixes bots.

Relax C++11 atomic ops memory order to acquire-release

Implement C++11 <atomic> operations

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

Merge pull request #1094 from anthrotype/fix-appveyor-msys2

appveyor.yml: try updating msys2 to fix failing mingw-w64-x86_64 build

appveyor.yml: try only updating msys2 core, not the rest of the packages

maybe it'll get a bit faster

appveyor.yml: pass --needed option to pacman

so it won't attempt reinstalling packages which are already installed

appveyor.yml: try updating msys2 to fix failing mingw-w64-x86_64 build

https://github.com/harfbuzz/harfbuzz/pull/1093#issuecomment-405201903
https://ci.appveyor.com/project/harfbuzz/harfbuzz/build/job/ky7lao1ii1bi7ew4#L71

Move _hb_alignof later again

It uses constexpr...

Make hb_vector_t 8 bytes smaller

Align NullPool and CrapPool to HB_VECTOR_SIZE

Fix return type of alignof() fallback

Minor

Minor

1.8.3

Fix shift

Put back include dependencies

Move more stuff from hb-private.hh to hb-dsalgs.hh

Move more stuff to hb-dsalgs.hh

Move hb_bytes_t to hb-dsalgs.hh

Remove hb_auto_array_t

Include more basic internal headers from hb-private.hh

Minor renamings of internal inline functions

Move hb_vector_t and hb_lockable_set_t to hb-dsalgs.hh

Move hb_stable_sort to hb-dsalgs.hh

Also, include hb-dsalgs.hh from end of hb-pricate.hh

Minor