Remove hardcoded ICU include paths.

ICU exports them using LOCAL_EXPORT_C_INCLUDE_DIRS.

https://android-review.googlesource.com/#/c/121311/

Merge pull request #76 from cpeterso/cpeterso/Wunused-function

Add #ifdef HB_USE_ATEXIT to fix -Wunused-function warnings

Add #ifdef HB_USE_ATEXIT to fix -Wunused-function warnings

Merge pull request #75 from cpeterso/cpeterso/Wunused-value

Fix hb_atomic_ptr_cmpexch -Wunused-value warnings

Fix hb_atomic_ptr_cmpexch -Wunused-value warnings

Add missing va_end()

Fixes https://github.com/behdad/harfbuzz/pull/74

[ft] Don't set font ppem

For discussion see:

  http://lists.freedesktop.org/archives/harfbuzz/2012-April/001905.html

Over time we have had added NO_HINTING all over the place in hb-ft.  Finish it off.
Not setting ppem on hb-font disables get_contour_point() calls which is good anyway.

See comments in the commit.

[ft] Add hb_ft_face_create_referenced() and hb_ft_font_create_referenced()

When I originally wrote hb-ft, FreeType objects did not support reference
counting.  As such, hb_ft_face_create() and hb_ft_font_create() had a
"destroy" callback and client was responsible for making sure FT_Face is
kept around as long as the hb-font/face are alive.

However, since this was not clearly documented, some clienets didn't
correctly did that.  In particular, some clients assumed that it's safe
to destroy FT_Face and then hb_face_t.  This, indeed, used to work, until
45fd9424c723f115ca98995b8f8a25185a6fc71d, which make face destroy access
font tables.

Now, I fixed that issue in 395b35903e052aecc97d0807e4f813c64c0d2b0b since
the access was not needed, but the problem remains that not all clients
handle this correctly.  See:

  https://bugs.freedesktop.org/show_bug.cgi?id=86300

Fortunately, FT_Reference_Face() was added to FreeType in 2010, and so we
can use it now.  Originally I wanted to change hb_ft_face_create() and
hb_ft_font_create() to reference the face if destroy==NULL was passed in.
That would improve pretty much all clients, with little undesired effects.
Except that FreeType itself, when compiled with HarfBuzz support, calls
hb_ft_font_create() with destroy==NULL and saves the resulting hb-font on
the ft-face (why does it not free it immediately?).  Making hb-face
reference ft-face causes a cycling reference there.  At least, that's my
current understanding.

At any rate, a cleaner approach, even if it means all clients will need a
change, is to introduce brand new API.  Which this commit does.

Some comments added to hb-ft.h, hoping to make future clients make better
choices.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=75299

Remove redundant check for FT_Face_GetCharVariantIndex

We require FreeType >= 2.8.3.  This symbol was introduced earlier
than that.

Fix FreeType version check

[ft] Open blob in READONLY mode

HB_MEMORY_MODE_READONLY_MAY_MAKE_WRITABLE is deprecated and fairly
useless now.

Avoid accessing layout tables at face destruction

"Fixes" https://bugs.freedesktop.org/show_bug.cgi?id=86300

Based on discussion someone else who had a similar issue, most probably
the user is releasing FT_Face before destructing hb_face_t / hb_font_t.
While that's a client bug, and while we can (and should) use FreeType
refcounting to help avoid that, it happens that we were accessing
the table when we didn't really have to.  Avoid that.

Fail blob creation if length overflows or is too large

Fail if blob start plus length overflows; or if blob length
is greater than 2GB.  It takes a while for fonts to get to that
size.  In the mean time, it protects against bugs like this:

  http://www.icu-project.org/trac/ticket/11450

Also avoids some weird issues with 32bit vs 64bit systems
as we accept length as unsigned int.  As such, a length of
-1 will cause overflow on 32bit machines, but happily
accepted on a 64bit machine.  Avoid that.

Assert that blob length doesn't overflow address.

This will crash now, if blob was created with wrong length.
Check for that coming next commit.

0.9.37

[otlayout] Mark variables that are C arrays as opposed to OT::ArrayOf

[otlayout] Avoid invalid access with Context format 3

[ot] Debug get_coverage

[ot] Print format in dispatch trace

Minor

[ot] Make sure all toplevel tables have tableTag

[sanitize] Improve debug output some more

[sanitize] Improve debug output

[indic] Allow ZWJ/ZWNJ before SM

In Oriya, a ZWJ/ZWNJ might be added before candrabindu to encourage
or stop ligation of the candrabindu.  This is clearly specified in
the Unicode section on Oriya.  Allow it there.  Note that Uniscribe
doesn't allow this.

Micro tests added using Noto Sans Oriya draft.

No changes in numbers.  Currently at:

BENGALI: 353725 out of 354188 tests passed. 463 failed (0.130722%)
DEVANAGARI: 707307 out of 707394 tests passed. 87 failed (0.0122987%)
GUJARATI: 366349 out of 366457 tests passed. 108 failed (0.0294714%)
GURMUKHI: 60732 out of 60747 tests passed. 15 failed (0.0246926%)
KANNADA: 951190 out of 951913 tests passed. 723 failed (0.0759523%)
KHMER: 299070 out of 299124 tests passed. 54 failed (0.0180527%)
MALAYALAM: 1048147 out of 1048334 tests passed. 187 failed (0.0178378%)
ORIYA: 42320 out of 42329 tests passed. 9 failed (0.021262%)
SINHALA: 271662 out of 271847 tests passed. 185 failed (0.068053%)
TAMIL: 1091753 out of 1091754 tests passed. 1 failed (9.15957e-05%)
TELUGU: 970555 out of 970573 tests passed. 18 failed (0.00185457%)

0.9.36

In hb_ot_collect_glyphs(), don't recurse to a lookup more than once

Otherwise, we might process a lookup thousands of times, with no
benefit.  This pathological case was hit by Noto Nastaliq Urdu Draft
in Firefox's code to determine whether space glyph is involved in
any GSUB/GPOS rules.  A test page is at http://behdad.org/urdu

See:
https://bugzilla.mozilla.org/show_bug.cgi?id=1090869

Move code around

Move macros around

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=84491

Fix misc warnings

Fixes https://github.com/behdad/harfbuzz/pull/51

Update test expectation for previous commit

[ft] Add NO_HINTING in a couple other places

[travis] Fix clang again

[travis] Fail build if coveralls fails

[travis] Only run coveralls under gcc

Help clang with gcov

[travis] Try to make coverage work with clang

[travis] Minor

Add README.md symlink to make github happy

[travis] Re-enable clang

Add build and coverage status links

[travis] Report matrix settings to help debugging

[travis] More coveralls

[travis] Give coveralls a hand

[travis] Disable clang for now

[test] Fixup test

[travis] Another try at coveralls.io

[travis] First try to hookup coveralls.io code coverage tracking

[travis] Unbreak

[travis] Minor

Create ragel-generated files in srcdir

Clean ragel-generated headers in maintainercleanfiles

Don't use tmp files in Makefile

Useful for code-coverage generation of the ragel-generated files
as they will now contain the correct .hh filename in them.

check-static-inits: Filter out zero-length sections

When compiling with -O0, zero-sized constructors were showing up
and confusing the test.

check-static-inits: check for static finalizers

Hide other bubble-sort

Hide bubble-sort!

[util] Fix ansi output when surface is empty

[ot-font] Add hb_ot_face_cmap_accelerator_t

[ot-font] Minor

[ot-font] Handle missing vertical metrics tables

[ot-font] Add metrics_accel_t

[ot-font] Start adding vertical support

Fix build

[arabic] Don't #include __FILE__

Fine!

https://code.google.com/p/chromium/issues/detail?id=406957

[arabic] Allow disabling win1256 fallback code

By defining HB_NO_WIN1256.

[ot] Prefer Windows platform cmap tables / accept all Unicode encodingIDs

Some fonts on the Mac ship with (0,1).

[win32] Avoid preprocessor warnings re macros

Fix check-symbols on OS X

[coretext] Hide feature_mappings

Simplify hb-object

Really fix clang build this time

Revert "Fix build on clang after previous commit"

This reverts commit 9c80cbc87f23040a334e30c750f64523d9ef10c8.

Fix build on clang after previous commit

Rewrite this==NULL checks to avoid undefined behavior

Fixes https://code.google.com/p/chromium/issues/detail?id=403594

Remove this==NULL check from face->reference_table()

Not supposed to happen, and apparently this is undefined in C++.
https://code.google.com/p/chromium/issues/detail?id=403594

Make hb_object_t members private

In preparation for fixing:
https://code.google.com/p/chromium/issues/detail?id=403594

Minor

0.9.35

[coretext] Add version guards for kCTLanguageAttributeName

[coretext] Pass buffer language to CoreText

[coretext] Minor

[hb-coretext] Fix cluster order of notdef runs in RTL text

[shape-plan] Fix typo!

The only effect is, if shaper_list was not NULL and no shaper was found,
we now don't insert anything into cache and return earlier.

[shape-plan] Better debug messages

[shape-plan] Simplify macro

Fix gcc warning

[shape-plan] Fix shape-plan caching with more than one requested shaper

Wasn't breaking out of loop, ouch!

http://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg1246370.html

[shape-plan] Add debug tracing

[coretext] Fix buffer resizing

Was very broken.  Now fixed and tested.

[coretext] Rewind scratch-allocated arrays when not needed anymore

[wince] Two more Windows CE fixes

Report has it that it builds (and works) now:

  https://codereview.qt-project.org/#/c/92087/

[coretext] Compare CGFont and PS name, if CTFont didn't match

See comments.

Fixes vertical text.  CoreText backend is in very good shape now!

Also see:
5a0eed3b50629be4826e4e9428f2c3255195395d
25f4fb9b56bb3f8bec821571c78f8829e40daa54

Fixes http://github.com/behdad/harfbuzz/pull/36

[coretext] Implement vertical shaping

Currently doesn't work though, we detect font fallback.  Apparently
matching on ct_font is not safe for this.  Looks like commit
25f4fb9b56bb3f8bec821571c78f8829e40daa54 wasn't enough after all.

[coretext] Remove unnecessary alt_size

Wasn't needed after a6b8dc87421de33746b0b14d86d2d1532aec02af.

[coretext] Use CFRunStatus

Assert that all runs had expected direction, and take hint for
non-monotone clusters.

[coretext] Minor

[coretext] Remove debug printf!

API: Do not clear buffer-flags in hb_buffer_clear_contents()

After 763e5466c0a03a7c27020e1e2598e488612529a7, one doesn't
need to set flags for different pieces of text.  The flags now
are something the client sets up once, depending on how it
actually uses the buffer.  As such, don't clear it in
clear_contents().

Tests updated.

Minor

[coretext] Pass buffer direction to CoreText

Have to use a CTTypesetter for this.