Remove zpp fallback code (always use Fast ZPP)

Squashed commit of the following:

commit 3e27fbb3d22b42d181e15c345f1c59a007b6b58c
Author: Andrea Faulds <ajf@ajf.me>
Date: S

Remove zpp fallback code (always use Fast ZPP)

Squashed commit of the following:

commit 3e27fbb3d22b42d181e15c345f1c59a007b6b58c
Author: Andrea Faulds <ajf@ajf.me>
Date: Sun Sep 11 19:14:37 2016 +0100

Keep dummy FAST_ZPP macro for compatibility

commit 8a7cfd00deaa4a3c5026c97580c49c886c72a5b4
Author: Andrea Faulds <ajf@ajf.me>
Date: Mon Sep 5 22:36:03 2016 +0100

Remove FAST_ZPP macro and plain zpp fallback code

show more ...

Revert "Do not prepend ? on nullables in ReflectionType::__toString()"

This reverts commit 8855a2ce76e8bfba1d2eea1345c765fde7a9a441.

Do not prepend ? on nullables in ReflectionType::__toString()

Better BC with 7.0.

Fix off-by-one in ReflectionType::__toString()

Review mistake...

Append \ to class name returned from ReflectionType::__toString()

Fixed bug #72846 (getConstant for a array constant with constant values returns NULL/NFC/UKNOWN)

ReflectionType improvements

Added ReflectionNamedType and updated ReflectionType::__toString()

Implement #38992: invoke() and invokeArgs() static method calls should match

We don't want ReflectionMethod::invoke() to simply ignore its first argument,
if the method to invoke is a st

Implement #38992: invoke() and invokeArgs() static method calls should match

We don't want ReflectionMethod::invoke() to simply ignore its first argument,
if the method to invoke is a static method. Instead we match its ZPP with
that of ReflectionMethod::invokeArgs(). Furthermore, we apply the DRY
principle by factoring out the code to a common helper function to prevent
inadvertent future divergence of the implementations of both methods.

As can be seen from the necessity to adapt some test cases, this causes a
BC break for some pathological cases. Therefore we apply this patch to PHP
7.1 only, which is still in beta phase.

show more ...

Fixed bug #72661 (ReflectionType::__toString crashes with iterable)

fix: bug72222 for PHP-5.6 reflection export of array consts

Add missed return replacing bail out

Fixed compilation warnings

Fixed compilation warnings

Replace zend_ce_error with NULL and replace more E_ERROR with thrown Error

Cleanup spurious changes.

Add Closure::fromCallable().

Add the ability to create closures from callable as part of RFC: https://wiki.php.net/rfc/closurefromcallable

fix #72209 (ReflectionProperty::getValue() doesn't fail if object doesn't match type)

Fixed bug #72174

Also fixes a memory leak if ::getValue() is used with __get().

Add missing update_constants in ReflectionClassConstant

Also fix indentation of __toString().

Revert code to use DUP instead of COPY

In a1c405e0c50c627cdd9a7695b4c7d644238b6b9b next to the actual fix
I have also switched some (effective) ZVAL_DUPs to ZVAL_COPYs. I'm
reverting

Revert code to use DUP instead of COPY

In a1c405e0c50c627cdd9a7695b4c7d644238b6b9b next to the actual fix
I have also switched some (effective) ZVAL_DUPs to ZVAL_COPYs. I'm
reverting this part as those were probably there for a reason
(presumably issues with non-atomic refcounting on ZTS).

show more ...

Fix usages of zend_update_constant_ex

If an in-place update in an external zval is performed, it needs
to incref'd beforehand, not afterwards.

- get rid of EG(scope). zend_get_executed_scope() should be used instead.
- ichanged zval_update_constant_ex(). Use IS_TYPE_IMMUTABLE flag on shared constants and AST, instead of "inline_change"

- get rid of EG(scope). zend_get_executed_scope() should be used instead.
- ichanged zval_update_constant_ex(). Use IS_TYPE_IMMUTABLE flag on shared constants and AST, instead of "inline_change" parameter.

show more ...

Removed "zend_fcall_info.function_table". It was assigned in many places, but is never used.

Remove IS_VAR_RET_REF flag

Instead decide whether a function returned by reference or by value
by checking whether the return value has REFERENCE type. This means
that functions retu

Remove IS_VAR_RET_REF flag

Instead decide whether a function returned by reference or by value
by checking whether the return value has REFERENCE type. This means
that functions returning by reference must always return a reference
and functions returning by value must not return a reference.

show more ...

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG(This) may store IS_OBJECT + $this or IS_UNUSED + "called_scope" (of course, "called_scope" may be NULL).
Some code might need to be adopted to support this change.
Checks (Z_OBJ(EX(This))) might need to be converted into (Z_TYPE(EX(This)) == IS_OBJECT).

show more ...