[fuzzing] Run fuzzing tests using hb-fuzzer as well

Revert "Fix hang in OOM situations"

This reverts commit f0599db761d7fc2d585d86e757a797f75ebc7499.

Commit abadc1717d997b69f987fdf1be9e12156d2d13d6 provides a better
fix for this.

Revert "Make sure we make progress in OOM situations"

This reverts commit 68b507a3c3c62c28c38e13fee733702bb703b6ca.

Commit abadc1717d997b69f987fdf1be9e12156d2d13d6 provides a better
fix for this.

1.1.0

Fix make distcheck

[travis] Only coverity-scan the coverity_scan branch

This essentially disables coverity-scan right now, until we find
a pattern to continuously submit branches there.

For background reasoning, see:
Fixes https://github.com/behdad/harfbuzz/issues/171

Add tests for previous two commits

To fully test what these are supposed to test, they should be run
against libharfbuzz-fuzzing.la instead of libharfbuzz.la, but for
now just record the files.

Fix another move_to assertion failure

If buf->idx is at end, don't set end past it...

Fixes https://github.com/behdad/harfbuzz/issues/173

Try to better handle OOM situations

Fixes assert fail in https://github.com/behdad/harfbuzz/issues/161
with libharfbuzz-fuzzing.

Try to fix gnome-continuous build fail

Enable building a bounded version of the library for fuzzing

test/fuzzing/hb-fuzzer links against libharfbuzz-fuzzing.so now.

Move things around

Merge pull request #163 from fanc999/msvc.src

Update the sources so they will compile under Visual Studio

Fix build on MSVC >= 2012

Use the DEFINE_ENUM_FLAG_OPERATORS macro in winnt.h on Visual Studio,
which defines the bitwise operators for the enumerations that we want to
mark as hb_mark_as_flags_t, which will take care of the situation on newer
Visual Studio (>= 2012), where the build breaks with C2057 errors as the
underlying types of the enumerations is not clear to the compiler when we
do a bitwise op within the declaration of the enumerations themselves.

Also disable the C4200 (nonstandard extension used : zero-sized array in
struct/union) and C4800 ('type' : forcing value to bool 'true' or 'false'
(performance warning)) warnings as the C4200 is the intended scenario and
C4800 is harmless but is so far an unavoidable side effect of using
DEFINE_ENUM_FLAG_OPERATORS.

hb-ot-shape-complex-arabic.cc: Fix build on Visual Studio

Visual Studio does not like declaring a enum variable within a for
statement, so fix the build by declaring the enum before doing the for
loop.

MSVC builds: Add fallback implementation for pre-2013 MSVC

Pre-2013 MSVC does not have scalbn() and scalbnf(), which are used in the
utility programs.  Add  fallback implementations for these, which can be
used when necessary.

util: Fix build on Visual Studio

Use the fallback implementation for lround() only on pre-2013 Visual
Studio, and ensure we are clear about the types of the parameters for
lround() and scalbnf(), since Visual Studio can be quite picky on
ambiguous parameter types.  Also, use g_ascii_strcasecmp() rather than
strcasecmp() as we are already using GLib for this code and we are
assured that g_ascii_strcasemp() is available.

For scalbnf() on pre-2013 Visaul Studio, a fallback implementation is
needed, but use another forced-included header for those compilers, which
will be added later.

Also use (char)27 on Visual Studio builds as '\e' is not a recognized
escape sequence, which will do the same thing.

[util] Fix crash when --output-file is specified but not --output-format

Merge pull request #167 from KonstantinRitt/unicode_cp_opt

Micro optimizations to UTF-16 and UTF-32 codecs

[arabic] Improve stretch length calculation

Err on the side of being too short, than too wide.  Reduces chance
of overlaps with neighboring glyphs.

[arabic] Don't stretch over cased letters

Addresses
https://github.com/behdad/harfbuzz/commit/6e6f82b6f3dde0fc6c3c7d991d9ec6cfff57823d#commitcomment-14248516

Revert "Don't process lookups if buffer is in error"

This reverts commit f92bd86cc8c11d262d1830c631cb7c63fc9d4bc8.

We don't want to be like cairo, where as soon as there's an error,
nothing works anymore.  So, lets process lookups as long as there's
no new memory needed.  That's also a model that hides fewer bugs.

Micro optimization to hb_utf16_t and hb_utf32_t ::prev()

Implement reverse lookup instead of re-using next()

Nano optimization to hb_utf16_t and hb_utf32_t ::next()

[fuzzing] Cap max buffer len at 128

Fix typo

Make sure we make progress in OOM situations

[fuzzing] Add test case for OOM

From https://github.com/behdad/harfbuzz/issues/161

Fix hang in OOM situations

Fixes https://github.com/behdad/harfbuzz/issues/161

Don't process lookups if buffer is in error

Limit buffer max size growth

https://github.com/behdad/harfbuzz/issues/161

[fuzzing] Add build system

[fuzzer] Add README

https://github.com/behdad/harfbuzz/issues/139#issuecomment-154202645

Implement SYRIAC ABBREVIATION MARK with 'stch' feature

The feature is enabled for any character in the Arabic shaper.
We should experiment with using it for Arabic subtending marks.
Though, that has a directionality problem as well, since those
are used with digits...

Fixes https://github.com/behdad/harfbuzz/issues/141

[ft] Fix extents with negative scales

Fixes https://github.com/behdad/harfbuzz/issues/162

Reserve a few scratch-flags bits for complex shapers

Add new shaper method postprocess_glyphs()

Unused currently.  To be used for Syriac stretch implementation.

https://github.com/behdad/harfbuzz/issues/141

[util] If font has color, generate PNG with color

Don't try zeroing marks by GDEF if there are no non-ASCII

This one is a hack, but should be ok.

Don't try zeroing mark attachments by Unicode if there's no non-ASCII

Actually commit changes for previous commit

Skip GPOS cursive / attachment adjustmnent if none happened

I'm starting to really like how free these new scratch_flags are.

Minor

Skip forming clusters if text is all ASCII

Move variable-sized struct member to end

Hopefully makes clang build happy

Optimize positioning for when h_origin is nil

Add font->has_...() to check for individual nil font funcs

Hopefully this wouldn't break any compiler.  There was some magic
moments to getting this to compile...

Minor reshaping of hb_font_funcs_t implementation

In anticipation of further changes.  No functional changes.

Comments

[ot] Remove font funcs that do nothing

[ot] Port ot font funcs to allocated object

[ft] Remove font funcs that do nothing

[ft] Port ft font funcs to allocated object

Separate nil font-funcs from parent-peeking font-funcs

API change: all fonts created with hb_font_create() now inherit from
(ie have parent) hb_font_get_empty().

Optimize positioning direction calculations

It makes the binary smaller AND faster.  Yumm!

Don't do fractions if buffer is ASCII-only

Optimize runs without Default_Ignorable's

Now that we have a buffer-wide scratch flags facility, use it to
optimize away a few passes.

Limit max edits during sanitize to 8

Used to be 100.  If a tabe needs more than one or two edits, it's
probably completely bogus...

Might help with speeding up fuzzing for
https://github.com/behdad/harfbuzz/issues/157

Minor

If font doesn't support U+2011, fall back to U+2010

Test passes now.

Untangle if/else waterfall

Add tests for hyphen fallback

U+2011 is <noBreak> equivaent of U+2010, so we should do the fallback
for it.  Currently fails.

[test] Drop hintings when subsetting fonts to record

Adjust the width of various spaces if font does not cover them

See discussion here:
https://github.com/behdad/harfbuzz/commit/81ef4f407d9c7bd98cf62cef951dc538b13442eb

There's no way to disable this fallback, but I don't think it would
be needed.  Let's hope for the best!

Fixes https://github.com/behdad/harfbuzz/issues/153

Generalize flags types

Normalize various spaces to space if font doesn't support

This resurrects the space fallback feature, after I disabled
the compatibility decomposition.  Now I can release HarfBuzz
again without breaking Pango!

It also remembers which space character it was, such that later
on we can approximate the width of this particular space
character.  That part is not implemented yet.

We normalize all GC=Zs chars except for U+1680 OGHA SPACE MARK,
which is better left alone.

[test] Support recording multiple lines of text in record-test.sh

Protect against possible invalid-memory access after OOM

Fix typo!

Ouch!  Fortunately that function was unused.

Add Unicode space category

Unused so far.

Make top-byte of unicode_props available to be used differently per-GC

Towards using top-byte of unicode-props for more things

Add _hb_glyph_info_is_unicode_mark()

Unused right now.

Tighten ccc-setting a bit and document it

Fix test expectation

[ot-font] Fix hmtx table length checking, *again*

Exactly the same problem that I fixed in
63ef0b41dc48d6112d1918c1b1de9de8ea90adb5

I rewrote the table checking yesterday in
67f8821fb25d9bd55719f5e29a582ae1af4b02b3
and introduced the exact same issue again. :(
Good thing we have ongoing fuzzing going now.  Was discovered
immediately by libFuzzer.  Thanks kcc!

https://github.com/behdad/harfbuzz/issues/139#issuecomment-153449473
Fixes https://github.com/behdad/harfbuzz/issues/156

[util] Fix option-parsing leaks

[util] In --debug mode, duplicate font data

This has the effect that the font data will end up in a memory
section malloc()ed exactly to its size.  This gives us better
valgrind detection of out-of-bounds access.

Previously, the font data was placed in a mmap()ed section or
GString-allocated area, which didn't have proper protections
at the end when running under valgrind.

[perf] Micro-optimize

Remove irrelevant comment

I tried moving the is_default_ignorable() function to an INTERNAL
function.  That made the binary size grow by 5k AND things got a
tad bit slower!

Minor

Combine unicode_props0/1 into a uint16

Slightly faster.  In prep for more changes.

[perf] Only call combining_class() for marks

Saves some time.  Also preparing for reusing the ccc byte for other stuff.

Make max context-length and max nesting level configurable

...at compile time.

[ot] Make bad-hmtx handling match FreeType

Also route fuzzing-related tests through hb-ot-font, to reduce dependency
on FreeType behavior for badly-broken fonts.  Fixes failing test with
FreeType master.

Use templates for making sure expression is constant

Update comments for removal of compat decompositions

Disable compatibility decomposition usage during normalization

Fixes https://github.com/behdad/harfbuzz/issues/152

Fix two more -Wshadow warnings

https://bugzilla.mozilla.org/show_bug.cgi?id=1215894

Fix one more -Wshadow warning

https://bugzilla.mozilla.org/show_bug.cgi?id=1215894

Fix a few more -Wshadow-local warnings

https://bugzilla.mozilla.org/show_bug.cgi?id=1215894

Fix one -Wshadow-compatible-local warning

From https://bugzilla.mozilla.org/show_bug.cgi?id=1215894

Avoid unnecessary cast to 64-bit

Fixes https://github.com/behdad/harfbuzz/issues/146

Or I think it should.

Add HB_FALLTHROUGH

Borrowed from https://bugzilla.mozilla.org/show_bug.cgi?id=1215411

Fix typo

Add BUILD.md to dist

Merge pull request #148 from ebraminio/inst

Add BUILD.md based on harfbuzz.org docs

Merge pull request #147 from ebraminio/dockerci

[ci] change to docker infrastructure

[ci] change to docker infrastructure

Add BUILD.md based on harfbuzz.org docs