Remove return value from llist apply functions

Unlike the hash apply functions, these do not return int.

Fix #79462: method_exists and property_exists incoherent behavior

Both functions are closely related, so should behave the same for wrong
input types, i.e. both should throw a TypeError.

Promote some warnings to Errors in Zend basic functions

Closes GH-5325

Make cast_object handler required

Avoid subtle differences in behavior depending on whether the
handler is absent or returns FAILURE.

If you previously set cast_object to NULL,

Make cast_object handler required

Avoid subtle differences in behavior depending on whether the
handler is absent or returns FAILURE.

If you previously set cast_object to NULL, create a handler that
always returns FAILURE instead.

show more ...

Clarify that return throws

Remove unnecessary NULL check

This is a required zpp parameter, it cannot be null.

Store aliased name of trait method

Currently, trait methods are aliased will continue to use the
original function name. In a few places in the codebase, we will
try to look up the a

Store aliased name of trait method

Currently, trait methods are aliased will continue to use the
original function name. In a few places in the codebase, we will
try to look up the actual method name instead. However, this does
not work if an aliased method is used indirectly
(https://bugs.php.net/bug.php?id=69180).

I think it would be better to instead actually change the method
name to the alias. This is in principle easy: We have to allow
function_name to be changed even if op array is otherwise shared
(similar to static_variables). This means we need to addref/release
the function_name separately, but I don't think there is a
performance concern here (especially as everything is usually
interned).

There is a bit of complication in opcache, where we need to make
sure that the function name is released the correct number of times
(interning may overwrite the name in the original op_array, but we
need to release it as many times as the op_array is shared).

Fixes bug #69180.
Fixes bug #74939.
Closes GH-5226.

show more ...

Change argument error message format

Closes GH-5211

Merge branch 'PHP-7.4'

* PHP-7.4:
Fix #63206: Fully support error/exception_handler stacking, even with null or inside the handler

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
Fix #63206: Fully support error/exception_handler stacking, even with null or inside the handler

Fix #63206: Fully support error/exception_handler stacking, even with null or inside the handler

Always push the current user_error/exception_handler to the stack,
even when it is empty,

Fix #63206: Fully support error/exception_handler stacking, even with null or inside the handler

Always push the current user_error/exception_handler to the stack,
even when it is empty, so restore_error_handler() always works as
expected.

The user_error_handler is especially temporarily empty when we are inside
the error handler, which caused inconsistent behaviour before.

show more ...

Make type error messages more consistent

Closes GH-5092

Convert zend_parse_parameters_none() to fast ZPP

I've done the conversion in those extensions where fast ZPP is predominant.

Use RETURN_THROWS() during ZPP in main, sapi, win32, and Zend

Merge branch 'PHP-7.4'

* PHP-7.4:
Similar problem (#79022) also exists in Interfaces
Fixed bug #79022 (class_exists returns True for classes that are not ready to be used)

Similar problem (#79022) also exists in Interfaces

Fixed bug #79022 (class_exists returns True for classes that are not ready to be used)

Make get_defined_vars() always return an array

GH-5025

Add missing ZPP checks

Closes GH-4878.

Merge branch 'PHP-7.4'

Fixed bug #78632

I'm going for a very conservative fix here, where the previous
logic is restored for the case where an object is passed to
method_exists(). We might want to check ag

Fixed bug #78632

I'm going for a very conservative fix here, where the previous
logic is restored for the case where an object is passed to
method_exists(). We might want to check against EG(scope) instead,
but this seems like a safer choice.

This means that behavior in PHP 7.4 changes only for
method_exists('C', 'privateMethodNotOnC'), which should be sensible.

show more ...

Merge branch 'PHP-7.4'

Report false for inherited private methods in method_exists()

These shadow methods only exist as internal implementation markers.
This mirrors the behavior of property_exists().

Throw TypeError from VM implementation of get_class()

The normal function already throws TypeError through the usual zpp
mechanism, but the VM implementation handles this manually and ha

Throw TypeError from VM implementation of get_class()

The normal function already throws TypeError through the usual zpp
mechanism, but the VM implementation handles this manually and has
not been updated.

show more ...

Generate arginfo from PHP stub files

Signature stubs for internal functions are specified in xyz.stub.php,
from which we generate actual arginfo structures in xyz_arginfo.h.
This fil

Generate arginfo from PHP stub files

Signature stubs for internal functions are specified in xyz.stub.php,
from which we generate actual arginfo structures in xyz_arginfo.h.
This file then needs to be included in the implementation appropriately.

Arginfo from stubs can be regenerated using scripts/dev/gen_stub.php.
However, this should also automatically happen when the stub file is
modified.

show more ...