Skip to content

Convert resources to objects in ext/sockets #5900

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
kocsismate wants to merge 8 commits into from
Closed

Convert resources to objects in ext/sockets #5900

kocsismate wants to merge 8 commits into from

Conversation

@kocsismate
Copy link
Member

@kocsismate kocsismate commented Jul 27, 2020

No description provided.

@kocsismate kocsismate marked this pull request as draft July 27, 2020 22:55
@Girgias Girgias mentioned this pull request Jul 27, 2020
Open
41 tasks
@Girgias
Copy link
Member

Girgias commented Jul 28, 2020

Compile error:

/home/vsts/work/1/s/ext/sockets/sockets.c:937:14: error: unused variable ‘php_sock’ [-Werror=unused-variable]
  937 |  php_socket *php_sock;
      |              ^~~~~~~~

nikic
nikic reviewed Jul 29, 2020
View reviewed changes
@kocsismate kocsismate added this to the PHP 8.0 milestone Jul 30, 2020
@kocsismate kocsismate marked this pull request as ready for review August 2, 2020 20:49
nikic
nikic reviewed Aug 2, 2020
View reviewed changes
kocsismate
kocsismate commented Aug 2, 2020
View reviewed changes
@nikic nikic force-pushed the resource-sockets branch from de6e36a to a0f43d9 Compare August 3, 2020 21:21
@php-pulls php-pulls closed this in 77172c2 Aug 3, 2020
@kocsismate kocsismate deleted the resource-sockets branch August 3, 2020 21:30
@jrfnl jrfnl mentioned this pull request Aug 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nikic nikic nikic left review comments

@bwoebi bwoebi bwoebi left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

PHP 8.0

Development

Successfully merging this pull request may close these issues.

4 participants

@kocsismate @Girgias @nikic @bwoebi