Oh well.  Again, who does fully understand what is right???

I don't like the hb_remove_reference() hack, but necessary.

Such that it carries lvalues inside.

Using decltype() instead.

Seprate namespace, cleaner, more clear.

Makes it nicer in pipelines.

inline variables are not C++11, so mark them unused...

For stricter enforcement.

Not all C++11 features are created equal when it comes to support...

warning: '_WIN64' is not defined, evaluates to 0 [-Wundef]

* src/hb-cff-interp-dict-common.hh: Use ull for unsigned int64_t

The llu suffix does not work for older Visual Studio versions
(pre-2013), but ull works for all the compilers that we attempt to
support.

* test/api: Fix build on pre-C99 compilers

Ensure variables are declared at the top of the block.

* src/hb-dsalgs.hh: Add specialization for hb_is_signed<> for __int8

Pre-Visual Studio 2010 does not consider __int8 (which is typedef'ed to
int8_t) to be equivilant to signed char, so the compiler cannot find the
corresponding hb_is_signed<> specialization that is needed.

The interesting thing is unsigned __int8 is considered to be equivilant
to unsigned char, so as the other types (short, int, long) that we look
for here, so only the specialization for __int8 is added here.

This will fix builds on Visual Studio 2008 at least.

We'll see which old clang versions this breaks...

We cannot take address hb_bytes_t direction.  We need to use the
newly added hb_addressof(), ala std::addressof().