[OTLayout] Minor refactoring

[OTLayout] Remove bogus caching of glyph property

Remember ZWNJ

To be used in upcoming changes.

Indent

[Indic] Improve base finding in final_reordering

Fixes 5 Malayalam failures!

MALAYALAM: 1048016 out of 1048334 tests passed. 318 failed (0.0303338%)

Add more dot-reph tests

[Indic] Track base position when reordering things

Ouch, how did things ever work without this?!  The added test that has a
dot-reph as well as a pre-base reordering Ra perfectly demonstrates the
bug (tested with Nirmala font from Win8 for example).  Testing suggests
that Win8 shaper has the *exact* same bug / behavior that we used to
have.  Odd.

[SEA] Don't zero any mark advances

Keep the logic simple, easier to explain to font developers.

Further adjust mark advance zeroing

This is a followup to 568000274c8edb5f41bc4f876ce21fcc8bdaeed8.
Looks like in the Latin shaper, Uniscribe zeroes all Unicode NSM
advances *after* GPOS, not before.  Match that.

Can be tested using DejaVu Sans Mono, since that font has GPOS
rules to zero the mark advances on its own.

[Indic] Fix Eyelash Ra with old Devanagari spec

[tests] Add tests for Devanagary Eyelash Ra

Currently broken with Sanskrit 2003 font.

[Indic] Apply 'blwf' before 'half'

This reverts 167b625d988b74572d6b2f646c285b666b650d49.  It didn't
matter before, but that's going to change with next commit.

[Indic] Apply 'vatu' before 'cjct'

This essentially reverts 1d6846db9ebf84561bb30a4e48c6c43184914099,
but that commit is from way back when.  We should be better
following the spec order now again.

Didn't mean to push this out

Ouch!

[Myanmar] Use master Indic table for syllable data

[tests] Add Cham sample

Improve checks for setmode()

As reported by Jonathan, OS X has setmode() that is something other
than what setmode() is on Win32.  So, limit invocation to Windows
platforms only.

Improve check for Windows platforms

Instead of checking for compiler, check for platform.

[SEA] Treat Consonant_Final like Consonant_Medial

Rename HB_OT_INDIC_OPTIONS env var to HB_OPTIONS

The Myanmar shaper now respects the uniscribe-bug-compatibility
option too.

Add South-East Asian shaper

Handles Tai Tham, Cham, and New Tai Lue for now.

Minor test reshufflings

[Indic] Make sure New Tai Lue works!

Adjust mark advance-width zeroing logic for Myanmar

Before, we were zeroing advance width of attached marks for
non-Indic scripts, and not doing it for Indic.

We have now three different behaviors, which seem to better
reflect what Uniscribe is doing:

  - For Indic, no explicit zeroing happens whatsoever, which
    is the same as before,

  - For Myanmar, zero advance width of glyphs marked as marks
    *in GDEF*, and do that *before* applying GPOS.  This seems
    to be what the new Win8 Myanmar shaper does,

  - For everything else, zero advance width of glyphs that are
    from General_Category=Mn Unicode characters, and do so
    before applying GPOS.  This seems to be what Uniscribe does
    for Latin at least.

With these changes, positioning of all tests matches for Myanmar,
except for the glitch in Uniscribe not applying 'mark'.  See preivous
commit.

[Myanmar] Add note re Uniscribe NOT applying 'mark'

Minor

[Myanmar] Fix handling of Punctuation and Symbol types

Testing with "clusters" now on par with testing without them.  15
failures both.

[Myanmar] Fixup handling of joiners and GB characters

[Myanmar] Prevent reordering between Asat and Dot below

Implemented as a hack for now.  Myanmar failures down from 23 to 15.

MYANMAR: 1123868 out of 1123883 tests passed. 15 failed (0.00133466%)

The remaining 15 cases are all where the syllable is wrong according to
the OpenType spec.  We insert dottedcircle.  Uniscribe fails to do that,
but it also fails to reorder the prebase-reordering medial-Ra.  So it
gets it wrong.

Shuffle test data around

Add Win8-style Myanmar shaper

Myanmar failures down from 51% to 0.00204648%!

MYANMAR: 1123860 out of 1123883 tests passed. 23 failed (0.00204648%)

Minor

[OTLayout] Respect lookup-flags skipping over non-mark glyphs

Before, when matching ligatures, we never skipping over base / liga
glyphs even if that was what the LookupFlags asked for.

Fixed now.  We carefully reviewed all instances of this, and tested with
Amiri as well as some Indic scripts, and are confident that this should
NOT break anyone's fonts.  It's also how Uniscribe does it, from what
we can tell.

[OTLayout] s/mark_skipping/skipping/

In aticipation of upcoming changes.

[Indic] Fix bug introduced in 8b217f5ac54aa0dcbba2dd6d59aa89dde33e56c2

Was breaking reph formation logic when the Ra is the only consonant.
Devanagari regression fixed.  Down to 57 failures again.  Ouch.

Improve normalization heuristic

Before, for most scripts, we were not trying to recompose two characters
if the second one had ccc=0.  That fails for Myanmar where U+1026
decomposes to U+1025,U+102E, both of which have ccc=0.  However, we do
want to try to recompose those.  We now check whether the second is a
mark, using general category instead.

At the same time, remove optimization that was conflicting with this.

[Let the Ngapi hackfest begin!]

Revert "Include config.h.in in tree"

This reverts commit 01013a0f5c58d5d54a37e5b4d6692e0bbd4baf80.

Conflicts:
config.h.in

Same argument as previous commit.

Revert "[Indic] Import ragel-generated Indic machine in git"

This reverts commit fab7a71f110ed646745d54b23284537c50058c21.

Conflicts:
src/hb-ot-shape-complex-indic-machine.hh

Keeping that generated file in-tree causes problems with processes like
tinderbox  that automatically fetch and build harfbuzz.  It's harder to
bootstrap harfbuzz now (as was previously), but I'm willing to give this
another chance and see how it goes.

Don't use $(ENV)

As reported by Peter Breitenlohner:

I think this is a very bad idea because ENV is used to specify a startup
file to be read by some/all shells.

Add color-tests automake option

Bug 60035 - intermittent make install failure on hb-version.h

Bug 59637 - check-exported-symbols.sh & check-internal-symbols.sh fail on mips/mipsel

Merge check-internal-symbols.sh and check-exported-symbols.sh

Really fix setmode this time

Thanks to Khaled for spotting it.

Fixup previous commit

Now, it looks like this breaks Cygwin again.  Have to figure out what's
going on.

Use setmode() instead of _setmode()

Looks like Cygwin / MSVC declare it that way, and it still works on
MinGW32 cross.

Bug 60053 - hb-common.cc:181:6: warning: ‘void free_langs()’ defined but not used

0.9.12

Minor

Minor

Fix build with Sun compiler

Revert "Minor"

This reverts commit 0a492357016bc9a614d2a726f2006c10af68ca58.

Enables code on more compilers.

Fix linking with non-gcc compilers

Fix residuals from fontconfig changes

0.9.11

Add atomic ops for Solaris

Based on fontconfig patch from Raimund Steger.

Fix crasher during multi-thread shaper data creation

Sample crash:

0  _hb_graphite2_shaper_face_data_destroy (data=0xffffffffffffffff)
    at ../../src/hb-graphite2.cc:129
1  0x00007ffff4271d7d in hb_graphite2_shaper_face_data_ensure (
    face=<optimized out>) at ../../src/hb-shaper-list.hh:35
2  hb_shape_plan_plan (shaper_list=<optimized out>, num_user_features=0,
    user_features=0x0, shape_plan=0xf7b490) at ../../src/hb-shaper-list.hh:35
3  hb_shape_plan_create (face=<optimized out>, props=<optimized out>,
    user_features=0x0, num_user_features=0, shaper_list=<optimized out>)
    at ../../src/hb-shape-plan.cc:108
4  0x00007ffff4272c93 in hb_shape_plan_create_cached (face=0x10cf2b0,
    props=0x11980d8, user_features=0x0, num_user_features=<optimized out>,
    shaper_list=0x0) at ../../src/hb-shape-plan.cc:283

[Indic] In old-spec shaping, don't move viramas around if seq ends with one

For example: u0c9a u0ccd u0c9a u0ccd with Lohit.  See:

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

[GPOS] If an Anchor offset is NULL, return false

If in a MarkPos table, a base has no anchor for a particular mark class,
return NULL such that the subsequent subtables get a chance at it.

Test case:
hb-shape ./EBGaramond12-Regular.otf ἂ --features="ss20","smcp"

Bug 58638 - Building fails with ICU due to not passing required flags

Rename hb_buffer_clear() to hb_buffer_clear_contents()

The previous name was clashing with harfbuzz.old.  There are systems
that need to link both...

Clash-free now again.

Remove a few unnecessary const's

Apparently helps with MSVC compilation.

0.9.10

[OTLayout] Always collect default language system in collect_lookups

Not sure if this is the most desired behavior.  It's the most easily
defined though.

[OTLayout] Fix feature iteration in collect_lookups

Previous logic was just wrong.

[OTLayout] Whitespace

[OTLayout] Handle required_feature_index in collect_lookups

[OTLayout] Fix various introspection issues with ClassDef's

As reported by Jonathan Kew.

[tests] Finish test-set.c

All passing now.

Protect sets in-error from further modication

Fixes test-set.c

[tests] Start adding tests for hb-set.h

Fails now.  Fixing.

[tests] Add set object to test-object.c

Minor

[Indic] Insert dottedcircle after a lone Malayalam dot-reph

Remove TODO that I don't intend to fix

Minor

Minor

Minor

[util] List supported output formats in --help output

[util] List supported output formats in hb-view

[util] Default to "text" output format in hb-shape

If you say:

  hb-shape font.ttf text --output-file out.txt

This was previously failing:

  Unknown output format `txt'; supported formats are: TEXT / JSON

Now we simply fallback to TEXT if no explicit format was requested.

[Indic] Reorder Malayalam dot-reph to after base

Test sequence is simple: U+0D4E,U+0D15.  The doth-reph should be
reordered to after the Ka.

https://bugzilla.redhat.com/show_bug.cgi?id=799565

Minor

Add Android.mk

Add build confirmation notice to configure

[TODO] Remove done items

Bug 58498 - Tests fail with gold linker on ARM

Use C++ linker if ICU is disabled

Bug 54948 - Undefined symbols: "operator delete(void*)" "operator
new(unsigned long)" "___cxa_pure_virtual"

[OTLayout] Limit alternate-location FeatureParams to 'size' feature

[OTLayout] Fix 'size' featureParams implementation

Looks at alternate location now.

[OTLayout] Fix tracing

[OTLayout] Towards correct FeatureParams handling

[OTLayout] Wire tag and list start all the way to Feature

To fix FeatureParam issues.  No actual fix yet, just plumbing.

[OTLayout] More 'size' feature sanity checking

We still don't look for the old incorrect place of the featureParams.
I'll wait till someone actually complains about it...

[OTLayout] Add FeatureParamsCharacterVariants struct

No API yet.

[OTLayout] Add UINT24 type

Make test-size-params write size in points

[OTLayout] Make hb_ot_layout_get_size_params() do some checks

[OTLayout] Change hb_ot_layout_get_params() API

And add implementation for StylisticSet UINameID.  No API yet.

[uniscribe] Enable Uniscribe backend only if it has ScriptShapeOpenType()

Bug 55494 - ScriptItemizeOpenType doesn't exists under Windows XP