Skip to content

[x] stream: async iterate don't register listeners if ended - #28996

Closed
ronag wants to merge 1 commit into
nodejs:masterfrom
nxtedition:stream-async-ended
Closed

[x] stream: async iterate don't register listeners if ended#28996
ronag wants to merge 1 commit into
nodejs:masterfrom
nxtedition:stream-async-ended

Conversation

@ronag

@ronagronag commented Aug 6, 2019

Copy link
Copy Markdown
Member

Don't register listeners if stream is already "ended".

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

This was referenced Aug 6, 2019
@addaleaxaddaleax added the stream Issues and PRs related to the stream subsystem. label Aug 6, 2019
@addaleax

Copy link
Copy Markdown
Member

Can you add a test?

@ronag
ronagforce-pushed the stream-async-ended branch from 2ad9390 to 1a8c9c2CompareAugust 9, 2019 16:04
@ronag

Copy link
Copy Markdown
MemberAuthor

closing for now

@ronagronag closed this Aug 28, 2019
@ronagronag changed the title stream: async iterate don't register listeners if ended[x] stream: async iterate don't register listeners if endedAug 28, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

streamIssues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ronag@addaleax