[uniscribe] Fix buffer allocation

Email from Jonathan Kew:

My cygwin build kept aborting on certain test words when run with the
uniscribe backend. Turned out this was caused by a bug in the allocation
of scratch buffers in hb-uniscribe.cc.

Commit 2a17f9568d9724e045d2c1d660e007f3acd747d9 introduced a new line

  ALLOCATE_ARRAY (SCRIPT_VISATTR, vis_attr, glyphs_size);

but it failed to account for this in the computation of glyphs_size
(the number of glyphs for which scratch buffer space is available),
with the result that the vis_clusters array ends up overrunning the
end of the scratch buffer and clobbering the beginning of the buffer's
info[].

AFAICS, the vis_attr array is not actually used, so the simple fix is
to remove the line that allocates it. (If/when we -do- need to use
vis_attr for something, we'll need to add another term to the earlier
calculation of glyphs_size.)

With this patch, the uniscribe backend runs reliably again.

JK

[Indic] Add Javanese config

We should add for other scripts too, send me the virama codepoint
and script name...

[OT] Collect requiredFeature only if features are not provided

As per Werner's report on the list.

Fix contextual lookup recursion indexing

See email thread "Skipping Control for Attaching Marks using OpenType"
from earlier this month.

Merge pull request #3 from LogosBible/master

Destroy lookups before blobs.

Destroy lookups before blobs.

'layout->gsub' may depend on data owned by 'layout->gsub_blob', so it must
be deinitialized before the blob is destroyed.

If variation selector is not consumed by cmap, pass it on to GSUB

This changes the semantics of get_glyph() callback and expect that
callbacks return false if the requested variant is not available, and
then we will call them back with variation_selector=0 and will retain
the glyph for the selector in the glyph stream.

Apparently most Mongolian fonts implement the Mongolian Variation
Selectors using GSUB, not cmap.

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

Note that this doesn't fix the Mongolian shaping yet, because the way
that's implemented is that the, say, 'init' feature ligates the letter
and the variation-selector.  However, since currently the variation
selector doesn't have the 'init' mask on, it will not be matched...

Skip over multiple variation selectors in a row

Fix misc warnings reported by cppcheck

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

Fix PKG_CHECK_MODULES usage

Oops.

Bug 65246 - Fix build without graphite2

Fix PKG_CHECK_MODULES usage

Bug 65246 - Fix build without graphite2

If we don't provide ACTION-IF-NOT-FOUND, configure aborts if it can't
find the libraries.  We handle that ourselves so we don't want the
macro to abort.

Minor

util/ansi-print.cc: Use fallback implementation for lround on MSVC

Unfortuately Visual Studio (still) does not support the C99 function
lround, so provide a fallback implementation for it.

Fix build for C89 compilers

Remove unnecessary stdint.h include

hb-uniscribe.cc: Re-enable build under Visual Studio

-Declare hinstLib at the top of block
-Fix the definitions of the typedefs of ScriptItemizeOpenType,
 ScriptShapeOpenType and ScriptPlaceOpenType

Fix ChangeLog regen

Update TODO

Fix test build

Minor

0.9.18

Fix dist

Generate harfbuzz-icu.pc

Currently it only works with ICU that has .pc files.  I'll
wait till someone complains before fixing it for icu-config
systems.

Build hb-icu into libharfbuzz-icu.so

Minor

Bug 55494 - ScriptItemizeOpenType doesn't exists under Windows XP

Fallback to older API if OpenType variants are not available.

Add --with-cairo

Add --with-freetype

Defaults to auto.

Default glib to auto again

Add --with-coretext

Defaults to no.

Add --with-uniscribe

Defaults to no.

Disable UCDN if glib is available

We only want UCDN if we don't have any other Unicode provider.
I'm going to remove ICU from the list of default Unicode providers
as part of moving hb-icu into its own library.  As such, the only
providers will be UCDN and glib.

Minor

Disable icu_le shaper

Will add a way to enable it later.

Remove icu-config fallback for icu_le shaper

Bug 64878 - compile error with ICU but not ICU

Bug 65053 - Add configure option to enable/disable ICU support

Disable hb-old shaper

Will add proper way to enable it later.

Bug 65054 - Add configure option to enable/disable Graphite2

Add --with-graphite2.  Defaults to off.

Remove hb-icu dependency from hb-icu-le shaper

Don't zero advance of mark-non-mark ligatures

If there's a mark ligating forward with non-mark, they were
inheriting the GC of the mark and later get advance-zeroed.
Don't do that if there's any non-mark glyph in the ligature.

Sample test: U+1780,U+17D2,U+179F with Kh-Metal-Chrieng.ttf

Also:
Bug 58922 - Issue with mark advance zeroing in generic shaper

[icu_le] Support non-BMP text

[icu_le] A bit better scaling

Merge commit '9d9e72e94e7914f82ce62a304e7242f79c13edaf'

[icu_le] Fix scaling

Seems to work...

Copy stdint.h boilerplate to ucdn

0.9.17

Minor refactoring

Arabic mark width-zeroing regression

Mozilla Bug 873902 - Display Arabic text with diacritics is bad
https://bugzilla.mozilla.org/show_bug.cgi?id=873902

Bug 64476 - Typo in hb_set_t.get_min()

Fixes previous commit's test.

[test] Add test for hb_set_get_min() bug

Failing now.

Bug 64476 - Typo in hb_set_t.get_min()

[OTLayout] Use is_inplace() when flipping buffers

[OTLayout] Merge / templateize apply_string()

[OTLayout] Minor

[OTLayout] Minor

[OTLayout] Add is_inplace() method to GSUB

[OTLayout] Refactor a bit more

Move code around

[OTLayout] Add hb_ot_layout_lookup_accelerator_t

[OTLayout] Add note about collect_glyphs() and recursive lookups

Add U+061C ARABIC LETTER MARK to Default_Ignorable

[OTLayout] Make MultipleSubst in-place for sequences of len=1

[OTLayout] Whitespace

[OTLayout] Minor refactoring

Minor

Minor

Minor

Minor

Renamed DEBUG to something else

Some infrastructures use DEBUG as a generic symbol.

Fix icu linking by requesting the library searchpath from icu-config

This is not ideal as we don't like -L/usr/lib in our linker line.
But this is only relevant to environments that don't have pkgconfig
files for ICU...

https://github.com/behdad/harfbuzz/pull/2

Make both old autoconf and new automake happy

Sigh..

Bug 64039 - undefined macro AM_PROG_AR

Minor

Minor renaming

Rename "pause" to "stage"

The compile() function is starting to become illegible...

Minor

Remove unneeded code

We always push a pause at the end such that each lookup falls in exactly
one pause_map_t.  Now, only if I can find a better name for that...

0.9.16

Hand-code bsearch in the hot inner loop.

Saves another 3 / 4 percent with Amiri.

Minor

[util] Unbreak --show-text / --show-unicode

Obssesive optimization

Not measurable by any means, but conceptually this is faster since
the mask matches more often than the digest.

Remove HB_DEBUG_SET_DIGESTS

Wasn't correct with the new combiner.  I should add it back somehow,
but for now the digests seem to be working very well...

Remove hb_set_digest_common_bits_t

Was unused.

Improve hb_set_digest_t

Make Amiri rendering faster a whopping 45% again!  Speends up pretty
much anything I tested.

Templatize hb_set_digest_lowest_bits_t filter

Add comment

Add hb_set_digest_combiner_t

Speed up Speed up hb_set_digest_common_bits_t calcs

Correctly this time.

Initialize set digests

We were not initializing the digests properly and as a result they were
being initialized to zero, making digest1 to never do any useful work.

Speeds up Amiri shaping significantly.

Print set-digest hit ratio with HB_DEBUG_SET_DIGESTS

Revert "Speed up hb_set_digest_common_bits_t calcs"

This reverts commit 3d1b66a35e1ab3be19335705f310b278d76d66d2.

The calculations were buggy.  It's not worth optimizing right now.

Fix typo in unused macro

[util] Add --num-iterations

Useful for profiling shapers.

[old] Speed up Unicode properties access in hb-old shaper

Just to be sure it's not punishing the old shaper performance
numbers.  Doesn't seem to have a measurable effect.

[git.mk] Update

Minor

0.9.15

[git.mk] Ignore ar-lib

Add note re Hangul shaping

Don't compose Hangul jamo

See thread "an issue regarding discrepancy between Korean and Unicode
standards" on the mailing list for the rationale.  In short: Uniscribe
doesn't, so fonts are designed to work without it.

Put back MemoryBarrier fallback implementation on MINGW32

This almost reverts 2761e8a632c14353f286708898be8df6ebad7407,
but only if under MINGW32, so it doesn't affect MSVC.