[util/hb-view] Fix rendering with all combinations of negative scales

[util] Minor

[util] Accept comma as well as space when separating components of args

Applies to --font-size and --margin.

Hopefully the scanf usage here doesn't have compatibility issues
(star being counted in the return value, etc).

Minor rename

Minor

Fixup

Fix warnings

Part of https://github.com/behdad/harfbuzz/pull/68

Err, instead of warn, if mutex / atomic / unicode funcs are missing

Hopefully this results in fewer badly built HarfBuzz integrations.

Fix unused var warnings

Minor refactoring to the atomics implementation

s/atomic_int/atomic_int_impl/ and s/atomic_ptr/atomic_ptr_impl/
to bring it in par with hb_mutex_impl_t, then re-introduce
hb_atomic_int_t as a wrapper around hb_atomic_int_impl_t.

In hb_reference_count_t, make it clear the non-atomic get and set
are intentional due to nature of the cases they are used in
(comparison to -1 and the debug output/tracing).

Minor

Allow implementing atomic and mutex ops in config

Motivated by
https://github.com/behdad/harfbuzz/pull/92

Minor

[test] Use /usr/bin/env python instead of /usr/bin/python

Bug 76494 - #!/usr/bin/python in testsuite

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

Merge pull request #93 from ebraminio/archpy3

Make hb_test_tools.py compatible with python 3

MSVC 2015 supports snprintf and not _snprintf

Move WinCE define to better place

Make hb_test_tools.py compatible with python 3

On ArchLinux, /usr/bin/python is linked to python 3 so
HarfBuzz `make check` is broken there.

This makes hb_test_tools.py compatible with python 3 while
no breaking it on python 2.

Fix VC++ /analyze warnings

out\debug\hb-buffer-deserialize-text.rl(47) : warning C6001: Using
uninitialized memory 'pos'.

Merge pull request #90 from khaledhosny/introspection-fixes

[bindings] Fix *_from_string functions

[bindings] Fix *_from_string functions

Without the element-type they will be getting garbage, at least with
Python.

0.9.40

Fix reverse_range() for empty range

Fixes coretext notdef loop consisting of all default_ignorable glyphs

https://code.google.com/p/chromium/issues/detail?id=464755

Fix previous commit

I misunderstood how which works.

Accept glibtoolize as libtoolize

Of course, we don't really run it, autoreconf does.  We just
err if neither is available.  glibtoolize is the name it is
shipped under on OS X.  Reported by Adam.

0.9.39

Fix hb-uniscribe build

Fix reverse_range() to only reverse alt array if positions are used

In hb-coretext, when we were using scratch buffer for book-keeping,
a reverse_range() caused by the notdef-insertion loop could mess up
our log_clusters.  Ouch!

Better error message if libtool is not installed

Fixes https://github.com/behdad/harfbuzz/pull/88

Clean up gtk-doc.make

Install git.mk in docs/

Update git.mk from upstream

Minor refactoring

Minor

Typo

Minor

[layout] Fix comparison of GlyphID and hb_codepoint_t

Before, the IntType::cmp functions providing this and was truncating
the hb_codepoint_t to 16bits before comparison.  I have no idea how
this was never discovered, and I'm too lazy to try to reproduce this
with Pango (which uses non-16bit codepoint numbers for missing glyphs).

Remove unused (and wrong as of a few commits ago) cmp() function

Minor

Minor

[layout] If lookup has only one subtable, move the forward loop down to subtable

I was hoping to see a nice speedup, but it resulted in a very minor one.

[layout] Add apply_forward / apply_backward

[layout] Remove unneeded return value from apply()

Minor simpilfy BEInt

Optimize IntType comparison to avoid branches for 16bit numbers

Minor simplify IntType

[layout] Minor

[layout] Minor

[layout] Don't check glyph props against lookup flags when recursing

Shouldn't be needed.  I have a hard time imagining this breaking any
legitimate use case.

[layout] Port sanitize() to use dispatch()

Needed some rework of Extension table.  Hopefully I got it right, and
the new template usage doesn't break any compilers...

Remove unused macro

[layout] Refactor Lookup::dispatch()

[layout] Refactor get_subtable()

Minor

[layout] Use dispatch() for add_coverage()

[layout] Add may_dispatch()

No functional change right now.

Make sanitize() a const method

This makes a lot of code safer.  We only try modifying the object in one
place, after making sure it's safe to do so.  So, do a const_cast<> in
that one place...

Minor

Remove unnecessary check in sanitize

[layout] Allocate iters in the context

Can be further optimized, but I think I didn't break anything.

Saves another 3% off Roboto shaping.

[layout] Move skippy_iter setup from constructor into init()

[lookup] Add skippy_iter.reset()

Towards reducing the cost of initializing skippy_iter()

[layout] Shuffle code around

[layout] Merge forward and backward iterators

[layout] Fix backward reject()

Has no functional effect since reject was never used with
match_glyph_data.

[layout] Remove some unnecessary checks in skippy

[lookup] Don't initialize skippy if coverage match fails

Currently:

  - Initializing skippy is very expensive,

  - Our lookup accelerator (using set-digests) can be very ineffecite,

As such, we end up many times initializing skippy but then failing
coverage check.  Reordering fixes that.

When, later, we fix our accelerator to have truly small false-positive
rate (for example by using the frozen-sets), then we might want to
reorder these checks such that we wouldn't calculate coverage number
if skippy is going to fail.

This shows a 5% speedup with Roboto already.

[layout] Use setter method to set c->lookup_props

[layout] Remove unused wrapper method

Add hb_frozen_set_t

I experimented with replacing use of hb_set_digest_t with this new
hb_frozen_set_t, hoping to get a huge speedup for busy lookups
(like kern lookup in Roboto), but I only got 6% speendup in Roboto
and 4% in NotoNastaliqUrduDraft :(.

Hide internals of lookup accelerators

[util] Add convenience "make lib" target

Merge pull request #85 from KonstantinRitt/define_inline

Fix build with MSVC on CE

Fix build with MSVC on CE

This code is C++ only. There isn't a single C++ compiler that fails to
understand the "inline" keyword, since it's required by C++98. Any
compiler older than C++98 is likely to choke on the template usage
further down, so this isn't necessary.

Moreover, the C++ standard says you cannot define macros.
[lib.macro.names] says "Nor shall such a translation unit define macros
for names lexically identical to keywords." -- technically, it's a
promise that the Standard Library headers won't do it, the wording means
that the entire translation unit won't do it, which implies no source
can do it.

MSVC complains about it:
fatal error C1189: #error : The C++ Standard Library forbids macroizing
keywords. Enable warning C4005 to find the forbidden macro.

Author: Thiago Macieira <thiago.macieira@intel.com>

[ft] Handle negative scales with vertical writing

[coretext] Don't generate notdef glyph for default-ignorables

As discovered on Chrome Mac:
https://code.google.com/p/chromium/issues/detail?id=452326

[coretext] Unbreak glyph positioning in presence of notdef runs

As discovered on Chrome Mac:
https://code.google.com/p/chromium/issues/detail?id=452326

This was originally broken in:

commit 5a0eed3b50629be4826e4e9428f2c3255195395d
Author: Behdad Esfahbod <behdad@behdad.org>
Date:   Mon Aug 11 23:47:16 2014 -0400

    [coretext] Implement vertical shaping

Minor optimization

Do not leak hb_language_t on hb_language_item_t destruction

Minor improvement to HB_SHAPER_DATA_DESTROY

[API] Add hb_buffer_add_latin1()

This is by no ways to promote non-Unicode encodings.  This is an entry
point that takes Unicode codepoints that happen to all be the first
256 characters and hence fit in 8bit strings.  This is useful eg in Chrome
where strings that can fit in 8bit are implemented that way, and this
avoids copying into UTF-8 or UTF-16.

Perhaps we should rename this to hb_buffer_add_codepoints8().  I'm also
curious if anyone would be really interested in hb_buffer_add_codepoints16().

Please discuss!

Fix hb_buffer_add_codepoints to actually NOT validate

Merge pull request #81 from KonstantinRitt/fixes/build/win8phone

winrt_buildfixes

Move some code around

Just to keep Windows specific workarounds in a single place.

Do not define MemoryBarrier on WinCE

There is a _HBMemoryBarrier() wrapper function that emulates
MemoryBarrier() behavior when it is not defined.

Fix build on WinRT

There is no environment (like WinCE) and the basic version
of InitializeCriticalSection is unsupported.

https://codereview.qt-project.org/#/c/92496/

0.9.38

[ft] Handle negative x_scale / y_scale

[coretext] Fix positioning of notdef

[coretext] Fix scaling

Before we were not accounting for possible differences in x_scale and
y_scale, as well as the signs of those.  All should be in good shape
now.

[coretext] Use vertical advance for notdef in vertical direction

Add convenience make target "make lib" in src/

Add README.python

[bindings] Add README.python

[util] Add --font-size to hb-shape

Also makes hb-view to use 8 bits of subpixel precision and shape at
requested size, instead of always shaping at upem and scaling results.

[util] Minor

When matching second glyph of kerning pairs, use bsearch

Roboto has glyphs (like 'F') that have 200 kerning pairs.
Add a handcoded bsearch instead of previous linear search.

This doesn't show much speedup though, apparently we spend the
bulk of the time somewhere before here.

[bindings] Minor

Merge pull request #77 from roozbehp/master

Change New Tai Lue shaping engine from SEA to default

Change New Tai Lue shaping engine from SEA to default

This is to reflect the UTC decision to change the encoding model of
New Tai Lue from logical to visual to be similar to Thai, Lao, and
Tai Viet: http://www.unicode.org/L2/L2014/14250.htm#141-C26

The visual encoding is already the current practice of encoding New
Tai Lue on the web anyway:
http://www.unicode.org/L2/L2014/14195-newtailue.txt

Fixes behdad/harfbuzz#66.