Skip to content

StreamSelectLoop: Fix erroneous zero-time sleep (2) - #93

Merged
jsor merged 2 commits into
reactphp:masterfrom
ivkalita:master
Apr 4, 2017
Merged

StreamSelectLoop: Fix erroneous zero-time sleep (2)#93
jsor merged 2 commits into
reactphp:masterfrom
ivkalita:master

Conversation

@ivkalita

Copy link
Copy Markdown
Contributor

Resolving #48.

Comment threadtests/StreamSelectLoopTest.php Outdated
public function testSmallTimerInterval()
{
/** @var StreamSelectLoop|\PHPUnit_Framework_MockObject_MockObject $loop */
$loop = $this->getMock(StreamSelectLoop::class, ['streamSelect']);

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.

::class requires PHP 5.5

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

@jsor, sorry, yes, just updated PR.

jsor
jsor approved these changes Apr 3, 2017

@jsorjsor 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.

Awesome! 👍

@jsor
jsor requested review from WyriHaximus and clueApril 3, 2017 19:53
@jsorjsor added this to the v0.4.3 milestone Apr 3, 2017

@WyriHaximusWyriHaximus 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.

👍

@clueclue modified the milestones: v0.5.0, v0.4.3Apr 3, 2017
clue
clue approved these changes Apr 3, 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.

Much appreciated! LGTM and should eventually also be cherry-picked to the v0.4.3 release branch 👍

@ivkalita

Copy link
Copy Markdown
ContributorAuthor

@clue, @jsor, @WyriHaximus, thank you! :)

@jsor

jsor commented Apr 4, 2017

Copy link
Copy Markdown
Member

@clue Filed #94 to add this patch to the 0.4 branch

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

@ivkalita@jsor@WyriHaximus@clue