Add TODO item

Don't return in void function

Would have been nice if gcc had warned...

Further simplify object handling

Add hb_object_header_t which is the common part of all objects

Makes way for adding arbitrary user_data support.

Move hb_reference_count_t to hb-private.h

Update Copyright headers

[hb-view] Accept numbers in feature tag name

Reported by Adam Twardoch.

Remove obsolete comment

Talking to Ryan Lortie, he thinks my comment doesn't make sense.

So I'm making the getter const.  Note that g_atomic_int_get()
casts that away itself, so we don't need to worry about that
(which kinda makes me uncomfortable actually).

Add ASSERT_STATIC_EXPR macro

Unused right now.

Cleanup hb_refrence_count_t

[TODO] Remove finished items

[API] Allow negative font x_scale/y_scale

I was reconsidering whether y should grow down, since all three/four
times I've used this API I was tricked and got that wrong in my use.
So I was very inclined to make y grow down instead of up.  However,
considering that the font space has y up and it would be very confusing
for callbacks to work against that, I decided that what I really want
is for the user to be able to set y_scale to a negative number to imply
that user-space y grows down.

Changing x_scale/y_scale from unsigned int to int allows that, and I've
made pango to use that instead of negating glyph y_offset later.  hb-ft
however still has y group up.  I *guess* that's how FreeType works?
I'm not sure, FreeType docs don't make this clear...

I'm happy with the resolution :-).

[icu] Remove big script switch(), rely on reverse-lookup

Move hb_reference_count_t from macros to inline methods

Rename all private sources and headers to C++ files

So we can liberally use the simple features of C++ that parts of the
codebase is already using.

Rename hb-blob.c to hb-blob.cc in preparation of more changes

[TODO] Remove done items

[API] Make hb_face_reference_table() return empty blob instead of NULL

The idea here is that:

  - Like pretty much all other API in harfbuzz, user does not have to
    check for NULL.

  - In any caller code, the case of missing table should be handled
    exactly the same way that a too-short table is handled.  Turning
    a non-existent talbe into a table of size 0 makes the user code
    safer.

[API] Rename hb_face_get_table() to hb_face_reference_table()

That correctly reflects the reference ownership transfer happening.

[TODO] Remove done items

[API] Remove hb_*_get_reference_count()

This was a bizzare piece of API that I inherited from cairo.  It has
been wrong adding them to cairo in the first place.  Remove them before
someone uses them!

[OT] Fix script to ot-script-tag conversion

Minor

[test] Add test for headers included from C and C++

[test] Add todo items

[API] font: move user_data before destroy()

This is the common convention for language binding tools.

[API] blob: move user_data before destroy()

This is the common convention for language binding tools.

[test] Rename test-types to test-common

[icu] Add two-way script conversion functions

Also optimizes the common-direction script lookup.

[glib] Add two-way script conversion functions

Also optimizes the common-direction script lookup to be an array lookup.

Remove verbose comments

unicode: Cleanup implementation

Cosmetic

[API] unicode: rework virtual functions for subclassing

Unicode data providers can now be subclassed, including support for
chain-up.  The interface should now be nicely bindable, as well.

Also fix glib unicode funcs that where broken after hb_script_t
changes.  Nicely caught by the test-unicode.c added in this commit.

[API] Rename hb_buffer_add_glyph() to hb_buffer_add()

[API] Add hb_buffer_allocation_successful()

Returns the error status of the buffer.

[API] Rename hb_buffer_ensure() to hb_buffer_pre_allocate()

The new name is self-documenting.

[API} hb_buffer_get_glyph_{infos,positions}: Add length out parameter

Return the length, whenever we return an array.  Makes it easier on the
language bindings.

[API] Use ISO 15924 tags for hb_script_t

This simplifies the code, reduces static data, and makes the design
more extensible to future additions of new scripts.

Change buffer default properties to invalid

This includes HB_DIRECTION_INVALID and HB_SCRIPT_INVALID.

The INVALID will cause a "guess whatever from the text" in hb_shape().
While it's not ideal, it works better than the previous defaults at
least (HB_DIRECTION_LTR and HB_SCRIPT_COMMON).

Move enum types to hb-common.h

[TODO] Add new item

Mark internal buffer variables private

Add hb_script_from_string()

Remove hb_buffer_clear_positions(), add hb_ot_layout_position_start()

[API] Remove hb_buffer_clear()

One should use hb_buffer_reset() really.

[TODO] Remove done items

Cosmetic

[TODO] Remove done items

Make hb_language_t typesafe

[test] Add more TODO items

Add hb_buffer_reset() and hb_buffer_set_length()

Minor

[test] Add todo items

[TODO] Add item re script iso15924 functions

[test] Test hb-buffer.h

[TODO] Add item re hb_buffer_reset()

[TODO] Add item re hb_buffer_resize()

[TODO] Add API item

Minor

[test] Actually hook up hb_script_t tests

I really shall find a way to automate the test enumeration.  Otherwise
it's too easy to add a test and wrongly hook it up.  Did it twice today.

Fold hb-language.[ch] into hb-common.[ch]

[test] Test hb_language_t

Make hb_language_from_string("") return NULL language

[teset] Test hb_script_t

Rename hb_category_t to hb_unicode_general_category_t

Make HB_TAG_CHAR4 private

It's just sugar.

Make hb_tag_from_string(NULL) return HB_TAG_NONE

Build fix

Move macros around

Rename HB_TAG_STR() to HB_TAG_CHAR4()

The problem with HB_TAG_STR() was that it expected a string of size 4
exactly, and unlike hb_tag_from_string() it doesn't pad the tag with
space characters.  So, the new name is more appropriate.

Add an in-tree test suite

Uses the glib testing framework.  Wrote unit tests for hb-common.h
types.

Add few more paratheses to the HB_TAG macro

Never hurts.

A few more ASSERTs

Remove warning message from public header file

Since we now assert thos in hb-private.h, the int types cannot be wrong.
(Except for when someone else includes hb-common.h in a very broken
configuration, but that's not our problem!)

Plus, we don't use inline in the public headers, so remove that too.

Assert int types

Add test directory

Update TODO items

Disable 'cswh' by default except for Arabic shaper

That better matches OpenType spec.  Note that we enable it for all
Arabic-shaper scripts.  Ie. we enable it by default for Syriac too,
but the SyriacOT spec does not require it.  I think this is a more
useful compromise than special-casing for Arabic script alone.

[hb-view] setlocale (LC_ALL, "")

For now we don't use anything from the locale, but we should default
to using $LANG, etc, if --language is not specified.  Right?

[hb-view] Rewrite --features parsing, with range support

The --features parsing handles errors now.  More importantly, it
allos limiting individual features to specific byte ranges.  The
format is Python-esque.  Here is how it all works:

  Syntax: Value: Start: End:

Setting value:
  "kern" 1 0 ∞ # Turn feature on
  "+kern" 1 0 ∞ # Turn feature off
  "-kern" 0 0 ∞ # Turn feature off
  "kern=0" 0 0 ∞ # Turn feature off
  "kern=1" 1 0 ∞ # Turn feature on
  "kern=2" 2 0 ∞ # Choose 2nd alternate

Setting index:
  "kern[]" 1 0 ∞ # Turn feature on
  "kern[:]" 1 0 ∞ # Turn feature on
  "kern[5:]" 1 5 ∞ # Turn feature on, partial
  "kern[:5]" 1 0 5 # Turn feature on, partial
  "kern[3:5]" 1 3 5 # Turn feature on, range
  "kern[3]" 1 3 3+1 # Turn feature on, single char

Mixing it all:

  "kern[3:5]=0" 1 3 5 # Turn feature off for range

[hb-view] Use cached hb-ft face creation

Avoids recreating the face the second time we call draw().

[hb-view] Add --debug

Frees all allocated memory before exiting.  Useful for valgrind run.

[hb-view] Support --features

Accepts values like:

  --features="-mkmk,aalt=2,calt=1,+cswh"

A minus sign means "=0", a plus sign means "=1".  Default is "=1".
A minus sign overrides an explicit value.

Sort options for clarity

Add script to/from ISO 15924 tag support

Also adds --script support to hb-view.

If a script tag is not known to us, we pass the ISO 15924 tag around.
Right now, the OT layer ignores that, but we can fix it to blindly
convert that to an OT script tag.

Let hb_shape() guess script and direction...

- Rename HB_SCRIPT_INVALID_CODE to HB_SCRIPT_INVALID

- Add HB_DIRECTION_INVALID

- Make hb_script_get_horizontal_direction() public

- Make hb_shape() guess script from buffer text (first non-common
  non-inherit script) if buffer script is set to HB_SCRIPT_INVALID (this
  is NOT the default.)

- Make hb_shape() guess direction from buffer script if buffer direction
is set to HB_DIRECTION_INVALID (this is NOT the default.)

- Make hb-view.c set INVALID script and direction on the buffer.

The above changes are meant to make hb-view fairly useful for uni-script
uni-direction text.  The guessing behavior however is NOT the default of
hb_shape() and must be asked for explicitly.  This is intended, because
the guess is not a suitable substitute to full-fledged bidi and script
segmentation.  It's just a testing tool.

[hb-view] Handle write_to_png errors

[hb-view] Link with -lm

Fix possible overflow

Default background color shall either be white or transparent

Definitely not black!

Add initial hb-view tool

Currently doesn't handle script or direction.

Fix #pragma message for MSVC.

__LOC__ was renamed to _HB__LOC__ in cd7555 but the corresponding
change wasn't made in hb-private.h.

Fix language extension matching

Don't use the m4/ dir

We just require people to have libtool, pkg-config, etc installed
when running autogen.sh

Fix 'make distcheck'

Bug 31174 - Update autotools configuration

Fix ChanContext backtrack matching with GPOS

Reported on mailing list by Keith Stribley and Khaled Hosny.

Remove unused realloc

We always allocate and grow str and pos together.

Fix ICU detection