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.

More sanitize cleanup

Minor cleanup of sanitize

Done with an audit of all sanitize()

Check for (impossible) overflow

Cleanup ContextFormat3 sanitize

Add comment

Minor

Remove integrity check in Tag sanitize

Serves no useful purpose.

Don't sanitize raw table data

That part is performed by individual table sanitize.

Add coment

Remove unnecessary casts

Avoid overflow in TableDirectory sanitize

Make casts more explicit

Make sure semicolon is expected after DEFINE_NULL_DATA()

Improve Subst/Pos SubTable access and sanitize

Simplify Extension offset now that our int types have no alignment

Sanitize shallow in Lookup since the generic SubLookup has no methods

Watch for overflow in Array sanitize

Rename const_sub_array to sub_array since all consts are implicit now

Cosmetic

Simplify Lookup sanitize

Change header comment

More cast simplification

Rename ConstCharP to CharP (overloaded now)

Cleanup de-const-casting during sanitize

Add more comments

Add comment

Add comment

Improve comment

Further simplify IntType struct defs

Rename const_array() to array() (overloaded)

Remove ArrayAfter, use StructAfter in place

Convert NEXT() and ARRAY_AFTER() macros to templates

Use templates for const char * casts

Const correctness

Use templates for defining int typess

Remove unused macro

Rename macros

Minor

Move macros around

Remove unused macros

GNOME Bug 613015 - [HB] Does not sanitize Device tables referenced from ValueRecords

WIP

Improve comments

Remove stale TODO item

Use a function template instead of struct template for Null

Simplify Tag struct

Cosmetic

Fix Class operator return type

Add couple consts to operators

Remove ASSERT_SIZE_DATA

Round instead of trunc

Fix warnings

Fix debug build

Fix leak when duplicating blob

Fix the mystery bug!

A couple bugs joined forces to exhibit the mystery behavior of
crashes / infinite loops on OS X / wrong kerning / invalid memory
access.  Pooh!

The bugs were involved:

  - Wrong pointer math with ValueRecord in PairPosFormat1

  - Fallout from avoiding flex arrays, code not correctly updated
    to remove sizeof() usage.

We strictly never use sizeof() directly now.  And the PairPos code
is cleaned up.  Should fix them all.  Bugs are:

  Bug 605655 - Pango 1.26.2 introduces kerning bug
  Bug 611229 - Pango reads from uninitialized memory
  Bug 593240 - (pangoosx) Crash / infinite loop with Mac OS X

We were also doing wrong math converting Device adjustments to
hb_position_t.  Fallout from FreeType days.  Should shift 16, not
6.  Fixed that too.

There's still another bug: we don't sanitize Device records
referenced from value records.  Fixing that also.

Typo

Cosmetic

[ft] Check stream->read instead of stream->base

The former is more robust.  See discussion on freetype-devel.

Make blob unlocking 64bit-safe

GNOME Bug 604128 - Applications crash when displaying Hebrew characters

Improve the alignment for NullPool

Use __attribute__((unused)) only with gcc 4 and later

See discussion at:
https://bugzilla.gnome.org/show_bug.cgi?id=610183

[GDEF] Fix bug in building synthetic GDEF

[TODO] Add kern/GPOS interaction

Fix compile with older FreeType

Indent

Revert "Fallback to 'kern' if no GPOS applied"

This reverts commit d740c8f78bcbe80a8bcd8a9468830a406da37cc0.

The change is wrong.  If there is a GPOS table, we should not use 'kern'.

Fallback to 'kern' if no GPOS applied