Skip to content
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.

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

[PHP 8.4] sockets constant list update #4154

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

devnexen
Copy link
Member

No description provided.

@devnexen
Copy link
Member Author

for #3872

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Please double-check that there are no tab characters used

<listitem>
<simpara>
Set the socket to a datagram congestion control protocol.
Available as of PHP 8.4.0 (NetBSD only)
Copy link
Member

Choose a reason for hiding this comment

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

Please add a full stop at the end of the "Available as of PHP 8.4.0." for all the constants

Suggested change
Available as of PHP 8.4.0 (NetBSD only)
Available as of PHP 8.4.0. (NetBSD only)

Comment on lines +2818 to +2819
Set the port range used for selecting a local port number.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Copy link
Member

Choose a reason for hiding this comment

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

Tabs usage?

Suggested change
Set the port range used for selecting a local port number.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Set the port range used for selecting a local port number.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)

Comment on lines +2830 to +2831
Use the default range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Copy link
Member

Choose a reason for hiding this comment

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

Ditto tabs

Suggested change
Use the default range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Use the default range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)

Comment on lines +2854 to +2855
Use a low range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Use a low range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)
Use a low range of port values.
Available as of PHP 8.4.0 (FreeBSD/NetBSD/OpenBSD only)

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