Skip to content

win, pipe: stop read for overlapped pipe#1822

Closed
bzoz wants to merge 1 commit intolibuv:v1.xfrom
JaneaSystems:bartek-pipe-read-stop
Closed

win, pipe: stop read for overlapped pipe#1822
bzoz wants to merge 1 commit intolibuv:v1.xfrom
JaneaSystems:bartek-pipe-read-stop

Conversation

@bzoz
Copy link
Copy Markdown
Member

@bzoz bzoz commented Apr 26, 2018

Adds CancelIoEx call when uv_read_stop is used on overlapped pipe. This fixes issues within nodejs/node#19377.

Adds CancelIoEx call when uv_read_stop on overlapped pipe.

Refs: nodejs/node#19377
@bzoz
Copy link
Copy Markdown
Member Author

bzoz commented Apr 26, 2018

Copy link
Copy Markdown
Contributor

@cjihrig cjihrig left a comment

Choose a reason for hiding this comment

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

LGTM. Please try this with the Node integration CI. Oops. I see that you did that :-D

cjihrig pushed a commit that referenced this pull request Apr 29, 2018
Adds CancelIoEx call when uv_read_stop on overlapped pipe.

Refs: nodejs/node#19377
PR-URL: #1822
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@cjihrig
Copy link
Copy Markdown
Contributor

cjihrig commented Apr 29, 2018

Landed in 0f53cd0. Thanks!

@cjihrig cjihrig closed this Apr 29, 2018
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.

3 participants