Use ValueError instead of exceptions in SPL extension

Use normal error in SPL for uninitialized objects

Use normal error in SPL for 'An iterator cannot be used with foreach by reference'

Error promotions in SPL

Warning to Error promotion and a Notice to Warning promotion to align
with the behaviour specified in the Reclassify Engine Warnings RFC.

Closes GH-6072

Merge branch 'PHP-7.4'

* PHP-7.4:
Add missing initialization check to RegexIterator::getRegex()

Add missing initialization check to RegexIterator::getRegex()

Accept zend_object in zend_read_property

Cleanup SPL instantiation code

Check dual_it validity in CallbackFilterIterator::accept()

Avoid accessing intern->u.cbfilter null pointer, though it's
harmless here.

Only set DIT type on successful initialization

Convert SPL fatal error to Error exception

Review the usage of apostrophes in error messages

Closes GH-5590

Remove no_separation flag

Don't allow separation in CallbackFilterIterator

As the name might suggest, this is a *filter* iterator. If you want
to have a *map* iterator, write one, or use a generator.

Remove proto comments from C files

Closes GH-5758

Add get_gc handle for object iterators

Optional handler with the same semantics as the object handler.

Don't use iterator_funcs_ptr if it is null

This avoids ubsan warnings. Alternatively we could always initialize
iterator_funcs_ptr for aggregates, instead of doing so only for
non-in

Don't use iterator_funcs_ptr if it is null

This avoids ubsan warnings. Alternatively we could always initialize
iterator_funcs_ptr for aggregates, instead of doing so only for
non-internal ones.

show more ...

Add ZVAL_OBJ_COPY macro

For the common ZVAL_OBJ + GC_ADDREF pattern.
This mirrors the existing ZVAL_STR_COPY API.

Remove redundant addref+dtor

Generate methods entries from stubs for ext/spl

Closes GH-5458

Add stubs for SPL iterators

Closes GH-5286

Don't use quiet zpp in RecursiveIteratorIterator ctor

Don't be a special snowflake, generate a standard TypeError here.

Define Stringable with __toString():string method

Eliminate uses of ZVAL_ZVAL and friends

Instead add RETURN_COPY(_VALUE) macros will the expected behavior.

RETURN_ZVAL doesn't make any sense since PHP 7, but has stuck
around,

Eliminate uses of ZVAL_ZVAL and friends

Instead add RETURN_COPY(_VALUE) macros will the expected behavior.

RETURN_ZVAL doesn't make any sense since PHP 7, but has stuck
around, probably because the alternative was to write directly to
the return_value variable.

show more ...

Use RETURN_THROWS() in various places