Clean up NO_INDEX

Fix awful confusion between lookup format and subtable format

As reported by John Daggett.

Fix tracing order

Don't fail sanitize on NULL data

Fix comment

Fix accessing tables from NULL pointer

Simplify unions

Remove unused macro

Change DEFINE_SIZE_VAR to DEFINE_SIZE_ARRAY

Cleanup DEFINE_SIZE_VAR2

Minor

Simplify array access

Fix check_struct to check min_size instead of sizeof

Further cleanup of sizeof

Remove CastP completely

Remove unnecessary casts

Make StructAtOffset take a pointer

Is safer.

Further cleanup of DEFINE_SIZE

Cleanup

Remove ASSERT_SIZE in favor of the safer DEFINE_SIZE_STATIC

Cleanup ASSERT_SIZE_VAR

Whitespace

Simplify DEFINE_NULL_DATA

Using ::min_size.

Start cleaning up get_size()

So we know when the size is static and when dynamic.

Remove the last of SANITIZE macros: SANITIZE_SELF

Remove SANITIZE macro

Cleanup Value casts

Remove SANITIZE_WITH_BASE

Remove SANITIZE_MEM

Rename check to check_range

Remove SANITIZE_ARRAY

Cosmetic

Minor

Minor

Simplify chaining

Remove APPLY_ARG_DEF and APPLY_ARG

Port apply to use hb_trace_t

Move context_length into apply_context

Move buffer into apply_context

Move layout_context into apply_context

Minor

I keep changing my mind about this

Remove SANITIZE_ARG_DEF and SANITIZE_ARG

Remove trace from sanitize_shallow()

Add hb_trace_t

Make sanitize_depth variable automatic and not passed through function args

Use function template for pass-thru argument

Make _hb_sanitize_*() methods of the context object

Remove SANITIZE_THIS

Rename SANITIZE_BASE to SANITIZE_WITH_BASE

Remove SANITIZE_OBJ

Remove SANITIZE_THIS2

Cosmetic

Remove SANITIZE_THIS3

Remove SANITIZE_BASE2

Remove SANITIZE2

Remove the NEUTER macro, move code to a method

Make internal method private

[main] Recognize Apple SFNTs

Comment new SFNT tags

Cosmetic: Rename HB_LIKELY/HB_UNLIKELY to likely/unlikely

Mark a couple functions as inline

Merge remote branch 'jrmuizel/master'

HB_UNUSED is unneeded on static inline functions

Include the tags from the Apple specification for TrueType fonts

Remove unused method

Whitespace

[GPOS] Speedup apply_value()

[ft] Disallow getting the entire font data with tag=0

Rename HB_GNUC_UNUSED -> HB_UNUSED

Define HB_FUNC for portability to non-gcc

[sanitize] Make debug code always available to the compiler

Such that we don't break debug build all the time.

Convert to uppercase in hb_ot_tag_from_language()

More contour point use

[gdef] Fix delta scale

[gdef] Fix rounding

[gdef] Implement getting contour point

Make sure mirroring is done in original direction

Reported by Jonathan Kew.

Use BCP47 private-use tags for OpenType tag language string mapping

Minor

Fix bug in hb_language_from_string()

Reported by Jonathan Kew.

Use const_cast

Add const

Rename apply_context -> context

Rename

[apply] Use a context object to reduce number of parameters passed around

Further simplify tracing

Simplify trace code

[sanitize] Make debug code always available to the compiler

Such that we don't break debug build all the time.

[gsubgpos] Make debug code always available to the compiler

Such that we don't break debug build all the time.

Fix compiler warning about value unused

[object] Make debug code always available to the compiler

Such that we don't break debug build all the time.

Minor

[blob] Make debug code always available to the compiler

Such that we don't break debug build all the time.

Fix build.  Ouch!

Fix debug build

[object] Actually handle malloc() failure

Caught by John Daggett.

Fix Device::get_size() calculation

Rename Var to Obj

Remove use of flexible arrays

Also remove wrong ASSERT_SIZE that would trigger only when not using
flexible arrays (which was the case on win32, but not with gcc).