Skip to content

Fix typo: Delete Concurrent#3620

Closed
msllws wants to merge 1 commit intohyperf:masterfrom
msllws:master
Closed

Fix typo: Delete Concurrent#3620
msllws wants to merge 1 commit intohyperf:masterfrom
msllws:master

Conversation

@msllws
Copy link
Copy Markdown
Contributor

@msllws msllws commented May 25, 2021

No description provided.

Copy link
Copy Markdown
Member

@sy-records sy-records left a comment

Choose a reason for hiding this comment

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

为啥删了这个呢

@msllws
Copy link
Copy Markdown
Contributor Author

msllws commented May 25, 2021

李老板让我删的
image

$concurrent = new Concurrent(10);

for ($i = 0; $i < 15; ++$i) {
$concurrent->create(function () use ($count) {
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.

应该是这个意思吧

- $concurrent->create(function () use ($count) {
+ $concurrent->create(function () {

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.

是的。。删掉 $count 就行了。。。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

不好意思打扰了

@limingxinleo
Copy link
Copy Markdown
Member

再改一下吧

@msllws msllws closed this May 26, 2021
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