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.

Minor on hb_blob_create_from_file

Add one more "unlikely" annotation and use explicit nullptr check for more consistency.

[subset] Add fuzzer test case that caused a timeout to the corpus.

Unify our pipe reader with the fallback reader (#1068)

And assign one bot to use the path always using NOMMAPFILEREADER token.

It's limited to 200mb so no more fun with using /dev/zero on hb-view!

Resolve ttx absolute path before use (#1075)

Treat - just as /dev/stdin and remove one extra file reader (#1065)

In Coverage iterator, bail out if table smells

In particular, if CoverageFormat2 has unsorted ranges, bail out.
Otherwise, 64k ranges of each 64k glyphs can DoS closure() method.

We can do the same for CoverageFormat1, but that one does not expose
the quadratic behavior, so, fine.

Support pipe and friends on hb_blob_create_from_file (#1061)

With this hb-view/hb-shape support pipes and possibly socket and named pipe
also, anything fails just on mmap.

We can later do the same for Windows also.

This however reveals two issues, the fact most of our bots don't have HAVE_MMAP and using
this instead the other fread/fopen reader can make failure on CI. I should look at them separately
this change however is very low risk I believe.

Revert file blob sanitization and index checking (#1062)

As https://github.com/harfbuzz/harfbuzz/pull/1059#issuecomment-397912812 we
like to still work with blobs that harfbuzz itself can't handle directly that are failing sanitization
currently apparently.

Verbose fail when something is wrong with hb-shape/hb-view input font file (#1059)

This checks if the blob isn't empty and uses `hb_face_count`
to see if the font file passes the simple font file sanitization
so can detect if the input is actually a font and checks also
whether input font-index is out of range.

Fix reading fonts from stdin (#1060)

We were passing the font path directly to freetype so rendering
was broken when we are getting the font from stdin.

This fixes it by using FT_New_Memory_Face instead.

This fixes:
* build/util/hb-view /dev/stdin text < font.ttf
* build/util/hb-view - text < font.ttf
* cat font.ttf | build/util/hb-view - text

but doesn't work on
* cat font.ttf | build/util/hb-view /dev/stdin text

which I will try to fix separately.

1.8.1

[docs] Enlist misc new symbols

Move hb-version.h generation to Makefile

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

Unfortunately 1.7.7 and 1.8.0 went out with the wrong hb-version.h contents.

[subset] Remove HB_SUBSET_BUILTIN

Just include hb-static.cc in libharfbuzz-subset.so source list as
well.  Those building it built-in will include hb-static.cc once
already.  No need for any gymnastics.

[substitute-closure] Rename function for clarity

Whitespace

[vector] Always 0-fill new items

Fix compiler warning re reordering of initializations

Style

[hb-set] Additional testcase for hb-set-intersect.

[hb-set] Don't shrink vectors until after processing their contents.

Fixes #1054.

Fetch the updated dwrite_1.h header from a better place

Following to https://ci.appveyor.com/project/harfbuzz/harfbuzz/build/1.0.1693/job/mfkjdhcdykjuqdfc

More fixes for SunStudio 12.6 build

Followup to https://github.com/harfbuzz/harfbuzz/pull/1053

Fix SunStudio 12.6 build (#1053)

[subset] Use REPLACEME instead of version.

[subset] Move variable declaration out of loop.

[subset] Add test cases for gsub closure in subsetting.

[subset] Add fuzzing of gsub closure to hb-subset-fuzzer.

[subset] Use gsub closure if ot layout is not being dropped.

[subset] Add drop_ot_layout setting to subset input.

[subset] add a new closure call to hb-ot-layout that can compute the closure over multiple lookups.

[set] Add is_subset

New API:
+hb_set_is_subset()

[subset] Add memoization of GSUB lookup closures.

Minorish

Compiler gymnastics

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

Move prime_for back into map

This was causing problem on systems without visibility when map was used
from both libharfbuzz and libharfbuzz-subset. Sigh.

https://ci.appveyor.com/project/harfbuzz/harfbuzz/build/1.0.1669/job/dey47nmff0770vp3

Convert Consonant_Initial_Postfixed to CONS_FINAL

Consonant_Initial_Postfixed was split off of Consonant_Succeeding_Repha,
so it should correspond to the same USE class, CONS_FINAL.

1.8.0

[ucdn] Update to Unicode 11

https://github.com/grigorig/ucdn/issues/19

Update to Unicode 11.0.0

UCDN is not updated yet.

Minor

[RELEASING] Update

1.7.7

[khmer] Fix Coeng vs Halant confusion

Test suite results are unchanged (34).

Fix hb_face_count build issue

I should've rebased #1002 before the merge, my bad

Add hb_face_count, a new API (#1002)

Simply, it returns the number of faces on a font blob.

To be used on hb-sanitizer tool but other clients also
can benefit from it.

[sinhala] Move reph to after post-consonants

Apparently this changed between Win 7 and Win 10.

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

To be continued in https://github.com/harfbuzz/harfbuzz/issues/1044

[test] Fix record-test to use gids not glyph-names

as those might not match what FontTools think of the glyphs if font
does NOT have glyph names.

Remove DISALLOW_* from vector, set, and map

Some of the build bots, still fail on this.  I suppose mine pass because
I enable C++11...

Anyway, remove these again.

Fix fallout from 975bdd5ef562e37655067b703b2b9ca7481f4985

Ouch!

[vector] Use Crap pool in push() as well

[vector] Keep success status

[vector] Whitespace

Another attempt at making every compiler happy...

Sigh.

[map] Move prime_for to hb-static

[set] Shrink page-map size again

Fix hb_vector_size_t

[subset] Have the subset fuzzer pull the text string to subset to from the end of the fuzzer provided data.

Add Codacy badge

Revert "Remove HB_DISALLOW_COPY_AND_ASSIGN"

This reverts commit ff92de766bf775bfdd3a01bda94de699180ff86a.

Revert that and remove ASSERT_POD. Let's see which bots are
unhappy with this configuration...

Use NullPool for _hb_set_nil and _hb_map_nil

[set] Minor

Rename in_error to !successful

Towards possibly using Null pool for some nil objects.

Fix a whitespace inconsistency

Null() and Crap() are only places that there's no space before '('...

Remove trivial HB_ATOMIC_INT_INIT()

Use 0 as inert object reference value instead of -1

Towards using Null object for nil objects.

Add const OffsetTo<> dereference

Unused, but now that we have CrapPool, implement it.

Fix symbol export issue of prime_mod on Alpine bot

Apparently our gcc-6.4.0 on Alpine Linux distribution doesn't like defining static const
inside a method, lets put that on outside the classes.

[map] Mark prime_for HB_INTERNAL

Trying to see if it fixes gcc-4.2 bots.

Add CrapOrNull

[subset] Switch to using hb_map_t inside of hb_subset_plan_t.

[map] Fix size calculation

Don't know why I thought I should subtract one there...

Remove HB_DISALLOW_COPY_AND_ASSIGN

llvm-gcc-4.2 bot had this problem:
hb-private.hh:812: error: initializer specified for non-virtual method 'void hb_vector_t<Type, StaticSize>::operator=(const hb_vector_t<Type, StaticSize>&) [with Type = hb_user_data_array_t::hb_user_data_item_t, unsigned int StaticSize = 1u]'

Removing the delete didn't work with a constructor. So, remove constructor.
Just disallow assignment.  Still better than nothing.

[map] Fix resize