Skip to content

Conversation

@Loghorn
Copy link
Contributor

@Loghorn Loghorn commented Oct 13, 2017

socket.getRecvBufferSize() and socket.getSendBufferSize() do not take any parameter

Checklist
Affected core subsystem(s)

doc

socket.getRecvBufferSize() and socket.getSendBufferSize() do not take any parameter
@nodejs-github-bot nodejs-github-bot added dgram Issues and PRs related to the dgram subsystem / UDP. doc Issues and PRs related to the documentations. labels Oct 13, 2017
Copy link
Member

@joyeecheung joyeecheung left a comment

Choose a reason for hiding this comment

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

I don't think this needs 48 hours to land

Copy link
Member

@joyeecheung joyeecheung left a comment

Choose a reason for hiding this comment

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

Wrong button..

joyeecheung pushed a commit that referenced this pull request Oct 14, 2017
socket.getRecvBufferSize() and socket.getSendBufferSize()
do not take any parameter

PR-URL: #16183
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
@joyeecheung
Copy link
Member

joyeecheung commented Oct 14, 2017

Landed in 34c47ed with commit messages fixed. Thank you for the contribution and next time, be sure to check out our commit message guidelines.

addaleax pushed a commit to ayojs/ayo that referenced this pull request Oct 15, 2017
socket.getRecvBufferSize() and socket.getSendBufferSize()
do not take any parameter

PR-URL: nodejs/node#16183
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
targos pushed a commit that referenced this pull request Oct 18, 2017
socket.getRecvBufferSize() and socket.getSendBufferSize()
do not take any parameter

PR-URL: #16183
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dgram Issues and PRs related to the dgram subsystem / UDP. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants