Destroy harfbuzz faces before freetype faces

HarfBuzz faces reference the FreeType faces.  Destroying FT faces first can cause crash in HB destruction code:

  https://bugs.freedesktop.org/show_bug.cgi?id=86300

Replace FT_Outline with ASS_Outline

Flip coordinate system in rasterizer

Fix incorrect closing of last drawing contour

Leave spline evaluation to rasterizer

Travis-CI: Coverity requires a notification email address

Or at least it looks like it does.

Fix broken memory leak check in legacy API

This was a check for the realloc() call, but the result was never
used in the success case - which makes no sense.

Doesn't really matter, because it's the legacy charset code which
hopefully is not used by anyone.

Found by Coverity.

Travis-CI: make Coverity scan work

Also add the obnoxious Coverity badge to the README
(thanks to torque for the reminder).

Travis-CI: minor changes

Strictly use the default compiler, but this probably has exactly the
same effect as using the "gcc" entry.

Do a parallel build.

Travis-CI: add coverity_scan branch to enabled branches

Travis-CI: add Coverity scan

Travis-CI: another attempt at making it work (2)

Travis-CI: another attempt at making it work

Switching to OSX, because homebrew is more uptodate than
Ubuntu Ancient LTS, which is what Travis uses on Linux.

Travis-CI: sigh

OK, so autogen.sh doesn't run configure.

travis-CI: configure needs to be generated

Add Travis-CI integration

More malloc checking

Don't crash if running out of memory when allocating per-glyph data
during shaping. Also some fixes to initialization.

The caller of ass_fontconfig.c/match_fullname() actually handles a NULL
return value correctly.

More malloc checking

In both cases, fallback to doing nothing if alloc fails.

More malloc checking

Use strndup() instead of malloc+copy.

Make all code deal with the possibility that ASS_Drawing.text can be
NULL (which can happen on allocation failure).

Skip fix_collisions() on malloc failure - the lines will overlap, but at
least libass won't crash.

Provide a strndup() fallback

This standard function is not available everywhere, so we detect it and
provide a fallback if missing.

Fix some of the recently added checks

Some more overflow checks

Deal with code-duplication bullshit

Much of outline_to_bitmap() is duplicated. What?

Add the overflow check to duplicated code too.

Check the overflow check for overflows

Seriously...

Checking for negative values might not be needed, but do it anyway.

Check bitmap allocation for overflows

This actually fixes #146.

The overflow check itself is obvious. Also, make ass_align() return an
unaligned value if aligning it would overflow. This is probably better,
as it for example makes the overflow check in the caller simpler.

Print debug message when loading a memory font

Split some parts of ass_render_event into functions

The split doesn't necessarily follow any logic; for the most part
it just moves each processing loop into its own function.

Check some allocations in strike-through code

Add another helper-macro for array allocation

Fix recently introduced overflow check (again)

Fix the bug

Commit 8536eaa was slightly broken: for some incomprehensible reason,
(w + 1) memory instead of w is needed. The missing space could lead to
memory corruption and crashes.

Simplify ass_utils.c code

Mainly cosmetic, but maybe some optimizations the compiler misses.

Clean up and improve .gitignore

Add * suffix to executable names for the sake of Windows,
where they get .exe extension automatically appended.

Use stdbool.h in another place

Also fix that ass_synth_init() could deref NULL.

Check for another overflow in blur code

Move apply_blur() to ass_bitmap.c

Put all code into one place, which makes it easier to follow.

Check against some overflows and allocation failures on blur

This still doesn't catch all cases.

Add ass_realloc_array()

Helps with overflow and allocation failure checking.

Check more mallocs

This is just a start and gets most easy ones.

Provide SIZE_MAX fallback

SIZE_MAX is in C99, but not in C89. It is in <stdint.h>, and is always a
macro, so this fallback should be pretty portable and cause no issues.

Check more allocations

Part of #146

Fix UB at left shifts of negative integers

Fix subpixel shadow shift

Only use the subpixel value when shifting bitmaps. Fixes #142

Load embedded fonts from memory correctly

Use LT_INIT for better autoreconf compatibility

Update version and changelog

Provide slightly more fine-grained control over style overrides

Silence compilation warning

This fixes the following warnings:

In file included from ass_cache.c:34:0:
ass_cache_template.h:52:0: warning: "BINSTRING" redefined
 #define BINSTRING(member) \
 ^
In file included from ass_cache.h:59:0,
                 from ass_font.h:58,
                 from ass_cache.c:28:
ass_cache_template.h:9:0: note: this is the location of the previous definition
 #define BINSTRING(member) \
 ^
In file included from ass_cache.c:36:0:
ass_cache_template.h:30:0: warning: "BINSTRING" redefined
 #define BINSTRING(member) \
 ^
In file included from ass_cache.c:34:0:
ass_cache_template.h:52:0: note: this is the location of the previous definition
 #define BINSTRING(member) \
 ^

Fix caching of combined bitmaps after complex shaping

Check FT_Outline_New return value

Makes #131 not crash. The function likely fails because the number of
points exceeds SHRT_MAX.

Correctly close last contour in case of overflow

Check for 0-point contours in rasterizer_set_outline()

Fixes libass#129.

Fix \fade(7-argument version) parse

Merge pull request #119 from MrSmile/fix-drawing-overflow

Fix integer overflow in FT_Outline

Merge pull request #116 from hadvaid/dragonfly

configure.ac: Treat DragonFly as the rest of BSDs

Fix integer overflow in FT_Outline

configure.ac: Treat DragonFly as the rest of BSDs

ass_parse: remove 2 unused variables

Fallout from the previous commits.

Simplify drawing text assignment

Use ass_drawing_add_chars for vector drawings

This not only provides a performance improvement but also
conveniently fixes the following issue: we used to interpret
backslash-escapes in vector drawings, but we shouldn't.

parse_tag: remove unnecessary malloc

Require closing '}' for override tags

Like VSFilter. '{' without a following '}' is just text, though
in vector drawing mode it still delimits individual drawings.

This also lets us nicely avoid '\0' hacks
in the \t override tag handler in parse_tag.

Introduce ass_drawing_add_chars for adding a whole string at once

Parse override tag arguments exactly like VSFilter 2.38

Also replace strtocolor in ass_utils with string2color
from ass.c, because that is more useful everywhere now.

Move (r)skip_spaces to ass_utils

parse_tag: merge \fs+, \fs-, \fs

parse_tag: split \fscx, \fscy, \fsc

Add a mechanism for selective style overrides

This adds 2 new API functions:

  ass_set_selective_style_override()
  ass_set_selective_style_override_enabled()

They can be used to force dialog text to use a specific ASS_Style. It
uses a fuzzy heuristic for that, and the quality of results may vary.
It does style overriding selectively and tries not to override things
that need explicit styling. The heuristic for that isn't set in stone
either, and can change with future libass versions.

Closes libass#88.

Merge pull request #100 from astiob/stroke

Use FT_Outline_EmboldenXY

Add BorderStyle=4, background color for text

This is somewhat similar to BorderStyle=3, but with a number of
differences. Mainly, this new BorderStyle doesn't create overlaps
within an event.

Closes #105.

Fix change detection when text is clipped to screen

The glyphs are clipped before combining, so the combined bitmap can
always have the same position, size, and address. This breaks the
change detection. Or at least I think that's what happens.

Remove bitmap restriding

It turns out we don't need this.

Use FT_Outline_EmboldenXY if available

Merge pull request #102 from grigorig/fribidi-errorhandle

shaper: add FriBidi error handling

shaper: add FriBidi error handling

It is unlikely, but FriBidi might not process a string correctly,
and we should handle that. Tested by making it fail always. This
should also fix some compiler warnings.

Skip useless memset() when copying a bitmap

Fix \1a, \2a, \3a, \4a with invalid argument

Reset to the initial color's alpha component, not red.

Reset clipping mode on every rectangle \clip

Prior to this fix, both of the following:
    \iclip(0,0,9999,9999)\clip(0,0,9999,9999)
    \iclip(0,0,0,0)\clip(0,0,9999,9999)
hid the whole picture in libass. The correct behavior
in both cases is to display the whole picture.

Make \be animatable

VSFilter has supported this since version 2.39.

Use the raw floating-point value of the \be argument in the animation
formula, like xy-VSFilter has done since version 3.0.0.45 (404301a3).

mystrtoll: don't truncate to int

Fix corner case: \move with identical start and end times

Implement fast quad-tree rasterizer in C and x86/SSE2/AVX2

Signed-off-by: Rodger Combs <rodger.combs@gmail.com>

Fix premature parser exit on }{ in drawing mode

Fixes #75.

Fixed headers in profile and test

Update README

Update version info and changelog

Initialize \pbo for each event

Regression in commit 1f5eb5eb.
Fixes issue #74.

Don't crash on \fscx0 with non-zero \blur

Regression in commit 98707694.

shaper: fix nasty sign bug in font functions

Fixes issue #68.

Merge pull request #66 from grigorig/drawing-floats

drawing: parse numbers as floats

drawing: parse numbers as floats

Parse values as float and convert into 26.6 fixed point. This actually
simplifies scaling a bit.

Fixes #63.

Remove dirty pixels from ASM be_blur output

A loop initializer was missing, so output started one row too early.
A loop condition check was missing, so output sometimes stopped
one column too late.

Also remove a couple of dead assignments.

Fix handling of top two pixel rows in be_blur_c

5dd56af2 replaced our implementation of \be with xy-VSFilter's.
This error is not present in the xy-VSFilter code; it was introduced
by accident in a merge that was later squashed as part of 5dd56af2.

Note: the new \be reads in but does not write out
the first and last row and column of pixels.

Remove incorrect declaration of HAVE_ALIGNED_STACK

Better fix for zero area outlines

Signed-off-by: wm4 <wm4@nowhere>
Commit 04b51c2d broke BorderStyle=3 with text that had spaces.

Fix API documentation of change detection

Merge pull request #61 from wm4/fix2

Fix change detection in presence of vector clips

Fix change detection in presence of vector clips

Fixes this test case:

Dialogue: 0,0:00:00.00,0:00:02.00,,,0,0,0,,{\fs50\pos(250,250)\iclip(m 0 0 l 400 0 l 400 220 l0 220)}Hello
Dialogue: 0,0:00:02.00,0:10:00.00,,,0,0,0,,{\fs50\pos(250,250)\iclip(m 0 0 l 400 0 l 400 230 l0 230)}Hello

The problem here is that the rendered output bitmap list itself does
not change, only its contents (due to the different vector clip).
ass_render_frame() will not set *detect_change correctly, and an
application using this flag (like mplayer or mpv) will not update
the screen as needed.

Fix this with a very cheap hack: always report a full change if
there's a vector clip. This is basically an emergency fix until
we have a proper way to detect the change.

Actually export ass_library_version symbol

Patch by chadr...@gmail.com.

Fixes libass#58, GC#117.