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

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

Fix syntax

Oops.

Add posix_memalign() fallback

Always compile (but not use) alignof() and round() fallback codes

Catches compile-errors in them better.

Fix alignof impl

Add fallback implementation for constexpr and alignof

Minor

Merge pull request #1091 from anthrotype/fix-gen-def-py

gen-def.py: pass headers as arguments so that msys2 can convert posix paths

CMakeLists.txt: don't pass header args as single space-separated string

let python's parse command-line args as usual

CMakeLists.txt: pass headers as arguments to gen-def.py; call using PYTHON_EXECUTABLE

src/Makefile.am: pass headers to gen-def.py as arguments, not env vars

gen-def.py: pass headers as arguments so that msys2 can convert posix paths

When one is not using the msys2 python, the header files that are passed in as environment
variable cannot be found.

https://ci.appveyor.com/project/fonttools/ttfautohint-py/build/1.0.65/job/rkremny4jjid9nl2#L803

This is because msys2 shell and make use POSIX paths (e.g. /c/Users/clupo/...)
whereas non-msys2 python.exe uses native Windows paths (e.g. C:\Users\clupo\...).

Msys2 will automatically convert command line arguments (but not environment variables) from
POSIX to Windows paths when calling a native win32 executable, so we pass the header paths
as arguments instead of environment variables.

This way the gen-def.py script can support both mingw python running in an MSYS2 shell, and
native win32 python.

Do not enforce a native direction of LTR for Runic

Fixes #481

Regenerate the USE table for Grantha and Bhaiksuki (#1090)

Completes #1037 and fixes #1035.

Disable vectorization

Disable vectorization for now.  To correctly use them, we should
use posix_memalign() to allocate them.  Otherwise, can cause
misaligned access.

https://bugs.chromium.org/p/chromium/issues/detail?id=860184

Fix warning

../../src/hb-ot-layout-gsubgpos-private.hh:391:18: warning: missed loop optimization, the loop counter may overflow [-Wunsafe-loop-optimizations]

Allow disabling vector_size use

Use CreateFileW explicitly and pass wchar_t to it (#1087)

[indic] Adjust left-matra repositioning and Halant,ZWJ sequence

From the new code (first paragraph is from the OT Devanagari spec.):

  /*   o Reorder matras:
   *
   *     If a pre-base matra character had been reordered before applying basic
   *     features, the glyph can be moved closer to the main consonant based on
   *     whether half-forms had been formed. Actual position for the matra is
   *     defined as “after last standalone halant glyph, after initial matra
   *     position and before the main consonant”. If ZWJ or ZWNJ follow this
   *     halant, position is moved after it.
   *
   * IMPLEMENTATION NOTES:
   *
   * It looks like the last sentence is wrong.  Testing, with Windows 7 Uniscribe
   * and Devanagari shows that the behavior is best described as:
   *
   * "If ZWJ follows this halant, matra is NOT repositioned after this halant.
   *  If ZWNJ follows this halant, position is moved after it."
   *
   * Test case, with Adobe Devanagari or Nirmala UI:
   *
   *   U+091F,U+094D,U+200C,U+092F,U+093F
   *   (Matra moves to the middle, after ZWNJ.)
   *
   *   U+091F,U+094D,U+200D,U+092F,U+093F
   *   (Matra does NOT move, stays to the left.)

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

Test case added with Adobe Devanagari.

Fix unused function '_hb_mapped_file_destroy' if no mmap

1.8.2

Disable .dfont code for now

The get_table imple was wrong, as table offsets in a dfont are
relative to the resource.  We were treating them as relative to
the big blob itself.  To be fixed.

Part of https://github.com/harfbuzz/harfbuzz/pull/1085

Fix trak table apply logic

In collaboration with Behdad

Support dfont font files (#949)

Improve nommap naming and use C style comments on create_from_file (#1084)

Adjust MarkBasePos heuristic in presence of MultipleSubst

From the issue:
"In this font, the virama,ya first forms a ligature, then decomposes back to
virama,ya. This causes those two to be marked parts of a MultipleSubst
sequence. When attaching the matra, we look for the first of the MultipleSubst
sequence because that's where we attach to (because of eg #740). In this case,
the first glyph in the MultipleSubst sequence is a mark, so we skip it and
attach to the base char before it."

Font in question is Nirmala UI from Windows 10. Test sequence:

  U+0926,U+094D,U+092F,U+0941

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

[indic] Tweak for old-spec Bengali and halant-ya-halant

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

Rebase and update USE overrides for Bhaiksuki Gap Fillers and Grantha Anusvara

[khmer] Fix infinite-loop in cluster merging

Indic shaper already had this check.  We removed it when forking
Khmer shaper by mistake.

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

armcc compatibility, don't use EINTR if doesn't exist

Fixes #1081

Minor on hb_blob_create_from_file, reuse ferror result

Oops

[ci] Don't fail on apt update and revive clang-O3-O0

Minor, rename blob to data on blob_from_file

Minor touches on hb_blob_create_from_file (#1079)

* Handle EINTR on fallback reader
* Increase fallback reader limitation size limitation to 2 << 28
* Ensure _O_BINARY does exist if MMAP is used on Windows
  (maybe superfluous but makes me more confident)

Make round() fallback portable to systems that do have round()

Makes compiling without HAVE_ROUND on systems that do have it work.