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

Disable use of thread_local

Clang build was failing with:

/usr/bin/ld: .libs/libharfbuzz_la-hb-blob.o: relocation R_X86_64_PC32 against undefined hidden symbol `_ZTH12_hb_CrapPool' can not be used when making a shared object

Instead of fighting it, just disable use of it.

[map] Not going to implement is_equal which is complicated

[map] Move prime_mod to header to avoid linkage issues in subset.so

[map] Fix copyright year

[map] Minor

[map] Don't return INVALID from get() just because in_error

[map] Fix bool use in C API

[map] More minor

[map] Make initial resize actually work

[map] Add clear(), is_empty(), and get_population()

[map] Minor

[map] Track population and occupancy separately

[map] Minor

Revert "[map] Return bool from set()"

This reverts commit face7cf55d4895ffca314c8448c0a749a26cc182.

Revert "[map] Make hb_map_set() return bool"

This reverts commit 7bf1980146cfc081b06264ac367b23ae9397adf1.

[map] Make hb_map_set() return bool

[map] Return bool from set()

[map] Implement operator[] for get()ting

[map] First try at implementing an integer-to-integer hashmap

Fully untested.

[blob] Use MAP_NORESERVE if available (#1039)

MAP_NORESERVE is not available on macOS for example so set the flag
to zero if not defined on the headers.

[set] Remove stale comment

Add HB_DISALLOW_COPY_AND_ASSIGN

Move pool definitions to hb-static.cc

Fix Uniscribe build

If a pointer type was passed to Null(), reinterpret_cast<> was
complaining about qualifiers being removed. Turns out I need the const on
the other side of "Type" to fix that.  Also remove unused const from
NullPool type.

Underflow protection in hb_vector_t

Mark CrapPool thread_local

Not sure if I like to keep it. For now, aim for correctness.

Add CrapPool

Common Regoin for Access Protection.  Like the NullPool, but writable.

Move code around

Add hb_vector_t::push(const Type &v)

Makes for cleaner code.

[ci] Revive the SunCC bot

* Adds libnsl to the bot as it is now needed apparently
* Disables hb-icu, apparently the newer ICU itself is not compatible with SunCC

Actually enable vectorized operations

Fixup for previous commit. Was accidentally turned off.

Another try at smart vectorization

Make clang happy. Also, don't use anonymous union.

Try enabling vectorization smartly

We'll see if this sticks to the bots.