Skip to content

Conversation

@kocsismate
Copy link
Member

No description provided.

Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good to me; maybe @devnexen also wants to have a look.

Copy link
Member

@devnexen devnexen left a comment

Choose a reason for hiding this comment

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

Nice, hopefully the stub part was not too tedious :-)

@kocsismate
Copy link
Member Author

Nice, hopefully the stub part was not too tedious :-)

Even though I used a regex for the conversion, the process was still very tedious because I had to merge the constants which are available both on Windows and other systems. :)

@kocsismate kocsismate merged commit 98e5c4e into php:master Aug 16, 2022
@kocsismate kocsismate deleted the const-sockets branch August 16, 2022 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants