Skip to content

set-error-handler.xml Remove the misleading statement#4536

Merged
vrana merged 1 commit intophp:masterfrom
mmalferov:patch-6
Apr 24, 2025
Merged

set-error-handler.xml Remove the misleading statement#4536
vrana merged 1 commit intophp:masterfrom
mmalferov:patch-6

Conversation

@mmalferov
Copy link
Member

For the method of the class as an error handler, the set_error_handler function may return a callable:

a) a string

b) a Closure

c) an indexed array (with the name of the class or an instance of the class in the first element, and the name of the method in the second one)

However, it is not limited to just "an indexed array with the class and method name".

I suggest removing the mention of the array as a return value, otherwise we will have to list every possible type of callable value :)

For the method of the class as an error handler, the `set_error_handler` function may return a `callable`:

a) a string

b) a Closure

c) an indexed array (with the name of the class or an instance of the class in the first element, and the name of the method in the second one)

However, it is not limited to just "an indexed array with the class and method name".

I suggest removing the mention of the array as a return value, otherwise we will have to list every possible type of callable value :)
@TimWolla TimWolla requested a review from arnaud-lb April 2, 2025 14:34
Copy link
Member

@arnaud-lb arnaud-lb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me!

@vrana vrana merged commit 21ce7d7 into php:master Apr 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants