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

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