Fix compile

NameRecord is not copy-constructible, so should be iterator of
const-reference.

Adjust is_cr_convertible

If To is const& then From doesn't need to be &.

[iter] Make hb_is_iterator_of() check is_convertible

Instead of is_cr_convertible.

[meta] Add hb_conditional<> and hb_is_convertible()

[subset] Switch building of glyph maps in subset plan to use iterators.

Add sink support for hb_hashmap_t and a reverse call to hb_pair_t.

Add hb_pair_t(,) macro as alternative to hb_pair_t<,>

Just so it's easier to use it in other macros.

Add hb_lidentity(), and rename hb_rvalue() to hb_ridentity()

[hdmx] Adjust to hb_iota() behavior change

Use hb_range() instead.

[enumerate] Fix hb_enumerate() len for step=0

[iter] Fix hb_zip() end condition

We should compare-equal to end if either iterator's end reaches,
not if both reach at the same time.  Fixes infinite-loop in test
which was happening after hb_enumerate() switched to using hb_zip().

[iter] Minor

[iter] Minor

[test-iter] Don't walk past end

That's not legal.

[iter] Allow negative step in hb_iota()

[iter] Minor

[iter] Bug fix

[iter] Fix

[iter] Take start value in hb_enumerate()

Also rewrite it via composition.

[iter] Add hb_range()

hb_range() is like Python range.  hb_iota() has slightly different API.
Ie. it takes a start, instead of end.

[meta] Add hb_int_max()

[iota] end -> end_ to not shadow

Merge pull request #1680 from n8willis/usermanual-obj

Usermanual: object-model chapter

Remove unnecessary template keyword

Should fix MSVC.

Remove unused var

[array] Add .copy()

Fix msan issue

hb_identity returns rvalue-reference if input is rvalue.  That, can leak
the reference and cause in bad access to temporaries after they are
destructed.  This is unfortunately unfixable given the desired
transparency of hb_identity :(.  Just don't use it with hb_map().

Move hb_invoke() back to hb-algs.hh

[hdmx] Touch up

[subset] WIP convert hdmx subsetting to use iterators.

[subset] Move hdmx to subset2.

[map] Return rvalues from keys()/values()

[map] Fix bots

Older compilers don't like calling iter() from return-type decltype()

../src/hb-map.hh:226:12: error: cannot call member function 'decltype ((((+ hb_array(((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::items, (((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::mask ? (((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::mask + 1) : 0))) | hb_filter((& hb_hashmap_t<K, V, kINVALID, vINVALID>::item_t:: is_real))) | hb_map((& hb_hashmap_t<K, V, kINVALID, vINVALID>::item_t:: get_pair)))) hb_hashmap_t<K, V, kINVALID, vINVALID>::iter() const [with K = const hb_serialize_context_t::object_t*; V = unsigned int; K kINVALID = 0u; V vINVALID = 0u; decltype ((((+ hb_array(((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::items, (((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::mask ? (((const hb_hashmap_t<K, V, kINVALID, vINVALID>*)this)->hb_hashmap_t<K, V, kINVALID, vINVALID>::mask + 1) : 0))) | hb_filter((& hb_hashmap_t<K, V, kINVALID, vINVALID>::item_t:: is_real))) | hb_map((& hb_hashmap_t<K, V, kINVALID, vINVALID>::item_t:: get_pair)))) = hb_map_iter_t<hb_filter_iter_t<hb_array_t<hb_hashmap_t<const hb_serialize_context_t::object_t*, unsigned int, 0u, 0u>::item_t>, bool (hb_hashmap_t<const hb_serialize_context_t::object_t*, unsigned int, 0u, 0u>::item_t::*)() const, const<anonymous struct>&, 0u>, hb_pair_t<const hb_serialize_context_t::object_t*, unsigned int> (hb_hashmap_t<const hb_serialize_context_t::object_t*, unsigned int, 0u, 0u>::item_t::*)() const, 0u>]' without object
     + iter()
            ^
../src/hb-meta.hh:58:41: note: in definition of macro 'HB_AUTO_RETURN'
 #define HB_AUTO_RETURN(E) -> decltype ((E)) { return (E); }
                                         ^

[map] Add .values() iterator

[map] Fix iter

[algs] Fix identity return type

Whitespace

[meta] Capture rvalue-references in is_reference / remove_reference

Minor

[map] tweak test-iter.cc

[map] add keys()

[map] add a test for iteration

[map] add iteration

[test] Add https://crbug.com/oss-fuzz/14641 testcase

As 503748d fix

initialize return param subr_num in popSubrNum

also snake_cased popSubrtNum and other surrounding function names

[name] Sanitize records for reals

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=14641

[iter] Rename hb_iter_t() to hb_iter_type<> and add hb_item_type<>

[map] Make .has() optionally return value

Make compiler happy with -Og

[iter] Accept pointer in hb_iter() and hb_iter_t()

[iter] Accept pointer in hb_map()

[serialize] LangSys subset->copy

Add hb_iota()

[iter] Actually fix previous commit

The iter objects shouldn't not be const.  D'oh.

[iter] For ref-qualified variants

Update README.md

Update README.md

[docs] add fuzzer instructions (courtesy of Garret)

[docs] add sample commands for test exec

[iter] Fix hb_sink() to accept rvalue

[iter] Default predicates to hb_identity instead of hb_bool

The bool conversion happens after predicate is called automatically.

Adjust hb_all/any/none

Add hb_match

[algs] Try f[v] in hb_get() as last resort

Fix a few more double-pomotion errors

Fix signed comparison on 32bit

Fix more double-promotion errors

WHy do only some of the clang bots catch this I have no idea :(.

Fix double-promotion warnings

Make it an error.

More tests

[algs] Sprinkle hb_min/max with hb-forward salad

Let's see if fixes MSVC fail.  Though, the error doesn't make sense to me.

  hb-blob.cc
c:\projects\harfbuzz\src\hb-algs.hh(166): error C2440: 'return': cannot convert from 'unsigned int' to 'unsigned int &&' [C:\projects\harfbuzz\build\harfbuzz.vcxproj]
  c:\projects\harfbuzz\src\hb-algs.hh(166): note: You cannot bind an lvalue to an rvalue reference
  c:\projects\harfbuzz\src\hb-algs.hh(174): note: see reference to function template instantiation 'T &&<unnamed-type-hb_min>::impl<T,unsigned int&>(T &&,T2) const' being compiled
          with
          [
              T=unsigned int,
              T2=unsigned int &
          ]

[iter] Add value and projection to hb_all/any/none

Allows for eg, checking all values equal 2: hb_all (it, 2).

[iter] Accept rvalues to hb_enumerate()

[meta] Move hb_invoke from algs to meta

[meta] Replace most hb_enable_if with hb_requires

They do absolutely same thing.  hb_requires is to encode constraints,
whereas hb_enable_if is for more conditional enabling.

[algs] Accept varargs in hb_min/max

Rename

Whitespace

Use universal references in hb_min/max

Remove MIN/MAX in favor of hb_min/hb_max

[serializer] Accept pointer & reference in more methods

[serializer] Minor

[meta] Fix addressof()

Whitespace

Add comment

[serialize] FeatureVariations subset->copy

Minor

[iter] Require lvalue in operators that return reference

Minor

[name] Clean up some more

[name] Clean up serialize() API

[name] Use iterators more

[serialize] Minor

Fix signed-comparison error on 32bit

Comment

[doc] Make header search more resilient

How stupid to scan all files... Sigh.

[ragel] Regenerate ragel-generated files using ragel 7.0.0.11 May 2018

[subset] Add TODO

[serialize] Enable bias assertion

[serialize] Use range-based loop

[name] Remove dead code