Skip to content

win, pipe: stop read for overlapped pipe - #1822

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

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

Conversation

@bzoz

@bzoz bzoz commented Apr 26, 2018

Copy link
Copy Markdown
Member

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

bzoz commented Apr 26, 2018

Copy link
Copy Markdown
Member Author

@cjihrig cjihrig left a comment •

Copy link
Copy Markdown
Contributor

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

cjihrig commented Apr 29, 2018

Copy link
Copy Markdown
Contributor

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