Add _MAX_VALUE to hb_ot_metrics_t (#1861)

minor

Add _MAX_VALUE to disabled baseline types enum

Replace 0x7FFFFFFFu in enums with HB_TAG_MAX_SIGNED

minor, comment out meta table in list till its use

Merge pull request #1851 from khaledhosny/fix-sbix-extents

Fix sbix glyph extents

Implement meta table parsing

[metrics] Fix _get_variation API to works with actual coord values

[metrics] _-prefix internal symbol

Fix warning on IBM compilers

Fixes https://github.com/harfbuzz/harfbuzz/issues/1852

Fix sbix glyph extents

* The ‘height’ needs to be negated since the API returns “distance from
  top to bottom side”.
* Similarly, the ‘y_offset‘ needs to be added to the height to get the
  ‘y_bearing’, since sbix’s offset is “the point in the glyph relative
  to its lower-left corner which corresponds to the origin” while
  ‘y_bearing’ is the “top side of glyph from origin”.

With these changes the sbix glyph metrics return values similar to other
tables, as they were otherwise unusable.

[metrics] Don't use metrics API in _common

As it is exposed with a different condition

Add hb_gdi_face_create API

Based on Konstantin Ritt work posted on mailing list

[metrics] minor, tweak comment

[docs] Minor

Revert hhea fallback to OS/2 to its reverse way

As searching number of hhea having fonts beats the number of OS/2
having ones in macOS 10.14.2

Bring back asc/desc abs logic used to be in hmtx table

Make HB_TINY builds work again by separating the always needed part

Merge _get_position_internal into _get_position

Define post table only when used

Merge ot-metrics-internal.cc with ot-metrics now that isn't needed in subset

Don't cache ascender/descender metrics

[doc] minor, improve hb-ot-metrics doc a bit

Merge pull request #1844 from ebraminio/hhea

Fallback hhea's ascender/descender to OS2

[os2] minor spacing tweaks

Fallback hhea's ascender/descender to OS2

[metrics] Fall back to hhea if OS2 metrics are empty

Reinstates previous logic, even if it might be unnecessary.

Add HB_NO_METRICS and fix HB_TINY build (#1839)

[ci][fuzzer] print valgrind failure if an error happened

[metrics] Fix weird use of xor

I believe that was a try to use one approach as fallback to other.  But
felt wrong.  Just believe what's in OS/2 table to be correct.

[ot-metrics] Touch up

Implement a simple API for fetching opentype metrics

Fixes https://github.com/harfbuzz/harfbuzz/pull/1432

Revert "[Myanmar] Prevent reordering between Asat and Dot below"

This reverts commit 1c8654ead41ca746d577549c92d2a41c594ab639.

[ci] Bring back -linux-arm64 bot

Let's see if 576065b has fixed it

Revert "Revert previous change"

This reverts commit b8e90ca1a10fbd472eda1aa8cc3797011da52356.

Works now.

[iter] Fix reduce type deduction

[ci] Disable -linux-arm64 bot

This is its failure https://circleci.com/gh/harfbuzz/harfbuzz/99864

Trying to fix like ee05627, interestingly, makes the bot and the
others to fail like this https://circleci.com/gh/harfbuzz/harfbuzz/99841

Revert previous change

Interestingly all of the bots disagreed with the change and the complain is... weird.

Improve syntax to make out linux-arm64 a little happy

Decided to apply is we did the same on other places however this won't
fix all of its complains

Downgrade -Wdeprecated-declarations to warning

Fixes #1834 at least till fix of #1829

[amalgam] Fix redundant-declaration warning/error

Fix C++20 compile warning on implicit capture of this with '=' default capture (#1833)

Happens when compiled with -std=c++2a, the fix just makes the captures explicit to resolve the issue. Just adding this in addition to = doesn't work in C++11.

src/hb-ot-layout-gpos-table.hh:737:18: warning: implicit capture of 'this' with a capture default of '=' is deprecated [-Wdeprecated-this-capture]
              { return (this+_).intersects (glyphs, valueFormat); })
                        ^
src/hb-ot-layout-gpos-table.hh:736:16: note: add an explicit capture of 'this' to capture '*this' by reference
    | hb_map ([=] (const OffsetTo<PairSet> &_)
               ^
                , this

[usp] define atfree callback only if used

[subset] For option "--unicodes",  add support for "*" to retain all code points

Revert "Update Graphite API to latest (#1215)"

This reverts commit e4e74c2751ac24178086cce2811d34d8019b6f85.

See https://github.com/harfbuzz/harfbuzz/issues/1829

[fuzz] Increase subset runner timeout for tsan bot

Now is flaky let's just increase and maybe investigate later

[fuzz] Don't fail when blob is empty

And enable more tests able to trig the issue.

Fix unintialized memory read in cmap subset (#1826)

Don't call memcpy when a table is empty

Force blob generation and memory check in hb-subset-fuzzer

Revert "Minor" - revert moving extern "C" definitions in-function

This reverts commit 62e60322cb9e18b3ee75f1b4a2a6d3069f587407 since it
breaks building HarfBuzz as part of Chromium.

Fixes https://github.com/harfbuzz/harfbuzz/issues/1821.

[font] Remove division when scaling

Yoohoo.  This seems to be precise enough!  Let's see if it sticks.
I'm asking Dominik to run this in Chrome test suite and report.

Fixes https://github.com/harfbuzz/harfbuzz/issues/1801

[font] Keep font-space to user-space multiplier

Part of https://github.com/harfbuzz/harfbuzz/issues/1801

Make face immutable in hb_font_set_face()

[ot-font] Optimize rounding

Part of https://github.com/harfbuzz/harfbuzz/issues/1801

The assumption that compiler optimizes "upem/2" to a shift only
works if upem is unsigned...  Anyway, spoon-feed the compiler.

Clusters are reversed based on the direction, not script

Fixes https://github.com/harfbuzz/harfbuzz/issues/1818

Docs: fix a typo in function name

hb_set_previous_range docs: fix presumed copy/paste error

Docs typo fix: slower → lower

[amalgam] Use it in cmake port and fix conflicts (#1812)

[amalgam] Include integration source files as well

Just for those that are normally built into libharfbuzz itself.

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

[amalgam] Rename hb.cc to harfbuzz.cc

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

Revert "Use constexpr to replace passthru_ bools"

This reverts commit c4aa10ebc8dc28b1f9c90af2ca2092a7535f8395.

Broke several compilers... Sigh.  The version without constexpr
didn't fully optimize out the unreachable code on clang.
So, revert it is...

Use constexpr to replace passthru_ bools

Sprinkle constexpr around

Being conservative.  Also not sure it makes any real difference
in our codebase.

Remove unused 'inline' specifier

[amalgam] Add hb.cc to git

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

Minor

[amalgam] Fix UCD issue

This actually makes it build now!

Part https://github.com/harfbuzz/harfbuzz/issues/1809

Keeping open to add tests, CI, etc.

[amalgam] Fix CFF

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

[amalgam] Finish fixing Indic-like shapers

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

[use] Remove Syriac features

This was non-standard, and unused anyway.

[amalgam] More Indic-like issues

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

[amalgam] More

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

[amalgam] Fix most duplicate-id instances in Indic-like shapers

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

Add make rule to build hb.cc

Part of https://github.com/harfbuzz/harfbuzz/issues/1809

Add ucd-table make target

add bimap test along with bug fix/tweaks

Merge pull request #1806 from carlo-bramini/master

Make harfbuzz working on all existing versions of Windows

[dwrite] Apply minor style improves

Fix error rised by GCC8+

Use lower case file name with #include

I tried to cross compile harfbuzz for Windows and an error was generated because `DWrite_1.h` was not found.
This happened because the filesystem is case sensitive and for this reason that include file was not found.
The right name of the file to be used is `dwrite_1.h`, with all letters not capitalized:
https://docs.microsoft.com/en-us/windows/desktop/api/dwrite_1/
I also verified in the installation of VS2017 with Windows Kit v10 and in that place it was also lower case.
So, in my opinion it should be better to change this.

Empty DIRECTWRITE_LIBS

Not used anymore since DWRITE is loaded dynamically.

Dynamically load DWRITE

Also checks if DWriteCreateFactory() has been executed successfully.

[config] Fix CoreText build with NO_AAT

Apply some minor improves on CFFIndex

minor, reuse StructAtOffset logic in StructAtOffsetOrNull

minor style improve in hb-cff-interp-common.hh

style fix for pylint complain

[config] Add links

[test] Make Unicode tests strict against internal UCD

https://github.com/harfbuzz/harfbuzz/pull/1799

Test at least one character per Unicode version

Merge pull request #1800 from harfbuzz/harfbuzz/cffnull_fix2

[CFF] fix for Subrs null ptr access

add test case file

alternate fix for https://crbug.com/971933

[config] Don't disable emoji sequences in HB_TINY

It makes sense to disable this code these if editing is not needed.
However, this is also necessary to correctly display emoji sequences
in right-to-left direction.  For that reason, don't auto-disable it.

[config] Compile out modified combining-class if HB_NO_OT_SHAPE

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

[emoji] Port generator to packtab

[config] Add HB_NO_EMOJI

Part of https://github.com/harfbuzz/harfbuzz/issues/1652

[config] Add HB_NO_SHAPER

Don't know who would want when why.  But makes sense to have.