Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

usleep(0) is valid #6961

Merged
merged 1 commit into from Nov 22, 2021
Merged

usleep(0) is valid #6961

merged 1 commit into from Nov 22, 2021

Conversation

LeSuisse
Copy link
Contributor

Similar to ef9858c.

usleep() only throws the value is < 0:

Uncaught ValueError: usleep(): Argument #1 ($microseconds) must be greater than or equal to 0

@LeSuisse
usleep(0) is valid
84d3276 
Similar to ef9858c.

usleep() only throws the value is < 0:
```
Uncaught ValueError: usleep(): Argument vimeo#1 ($microseconds) must be greater than or equal to 0
```
@weirdan weirdan added the release:fix The PR will be included in 'Fixes' section of the release notes label Nov 22, 2021
@weirdan weirdan merged commit 9943efb into vimeo:master Nov 22, 2021
@weirdan
Copy link
Collaborator

weirdan commented Nov 22, 2021

Thanks!

@LeSuisse LeSuisse deleted the usleep-0 branch November 22, 2021 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@weirdan weirdan weirdan approved these changes

Assignees
No one assigned
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@LeSuisse @weirdan