Skip to content

Conversation

@jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Oct 22, 2025

I vaguely remember these signatures are normally updated automatically from the stub files ? Not sure what happened here, but this one is clearly wrong.

Page: https://www.php.net/manual/en/function.stream-context-set-option.php

Signature as per the stub: https://github.com/php/php-src/blob/40a42cffd81b7f01fed25b3f7d02cb4d2ead700d/ext/standard/basic_functions.stub.php#L3384C78-L3384C103

I vaguely remember these signatures are normally updated automatically from the stub files ? Not sure what happened here, but this one is clearly wrong.

Page: https://www.php.net/manual/en/function.stream-context-set-option.php

Signature as per the stub: https://github.com/php/php-src/blob/40a42cffd81b7f01fed25b3f7d02cb4d2ead700d/ext/standard/basic_functions.stub.php#L3384C78-L3384C103
@TimWolla
Copy link
Member

TimWolla commented Oct 26, 2025

I vaguely remember these signatures are normally updated automatically from the stub files

They can be updated automatically, but it's still a manual process to start the automated update.

@TimWolla TimWolla merged commit e2d1e1f into php:master Oct 26, 2025
2 checks passed
@jrfnl jrfnl deleted the feature/stream_context_set_option-fix-param-name branch October 26, 2025 18:39
@jrfnl
Copy link
Contributor Author

jrfnl commented Oct 26, 2025

They can be updated automatically, but it's still a manual process to start the automated update.

And this may be one of the functions which is exempt from the auto-update anyway as it's one of the ones with a (deprecated) alternative signature.

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.

2 participants