Skip to content

StreamSelectLoop: Fix erroneous zero-time sleep - #49

Closed
joshdifabio wants to merge 1 commit into
reactphp:masterfrom
joshdifabio:patch-1
Closed

StreamSelectLoop: Fix erroneous zero-time sleep#49
joshdifabio wants to merge 1 commit into
reactphp:masterfrom
joshdifabio:patch-1

Conversation

@joshdifabio

Copy link
Copy Markdown

See #48.

@clue

clue commented May 20, 2016

Copy link
Copy Markdown
Member

Nice find 👍

clue
clue approved these changes Jan 9, 2017

@clueclue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@joshdifabio The changes LGTM 👍 Can you add some tests for this?

jsor
jsor approved these changes Feb 4, 2017
@clue

clue commented Feb 8, 2017

Copy link
Copy Markdown
Member

Ping @joshdifabio, what do you think about adding a test here?

Also, may I ask you to rebase this on the new "0.4" branch so we get this in for the upcoming v0.4.3 release? :shipit:

@clueclue added this to the v0.4.3 milestone Feb 8, 2017
@clueclue changed the title Resolve erroneous zero-time sleepStreamSelectLoop: Fix erroneous zero-time sleepFeb 8, 2017
@joshdifabio

Copy link
Copy Markdown
Author

I can't see an easy way to test this, but I'm also very busy and this isn't near the top of my priority list. Please feel free to close this PR and make the change yourself if that's easier.

@jsor

jsor commented Apr 4, 2017

Copy link
Copy Markdown
Member

Superseded by #93

@jsorjsor closed this Apr 4, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@joshdifabio@clue@jsor@WyriHaximus