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).

Cleanup hb_face_create_for_data()

Allow get_table() to return NULL

Typo

Remove lock_instance()

Add different casts from pointer and ref to avoid bugs

Remove GET_FOR_DATA macros

The major-version check is now handled by sanitize.  If major
doesn't match, we reject and fall back to the Null object.

Add TODO

Zero glyph metrics before calling user callback

More warning fixes

Remove unused parameter

Fix compile warnings (HB_GNUC_UNUSED)

No need to keep blob in sanitize context

When sanitizing, delay making writable

Before, as soon as we needed to make an edit, we tried to make the blob
writable inplace.  That grows code unnecessarily though.  We can simply
fail, make writable, then start again.  That's indeed what the fallback
was doing anyway.

[blob] Fallback to copying if mprotect() fails

Cleanup OpenTypeFontFile

Properly define separate structs for TTCHeader and TTCHeaderVersion1

Cleanup Extension lookups

Two things:

1. Allow nested Extension lookups.  The offset is always positive, so
it can't loop circularly.

2. Move the check for all Extension subtables having the same lookup
type to the correct place.  Before it wasn't really working.

Add comment re bsearch effect on sanitize

Cosmetic

Convert the last set of cast macros to templates

Remove unused macro

Cleanup Extension sanitize()

Fail sanitize on major version mismatch

We handle major-version differences via get_for_data(), so sanitize
should never see a major version mismatch.