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

asyncio: Use default sleep / ensure_future implementations when possible #1376

Merged

Conversation

zmedico
Copy link
Member

@zmedico zmedico commented Aug 31, 2024

When a loop argument is not given, use the default asyncio sleep / ensure_future implementations and avoid unnecessary _wrap_loop usage.

Bug: https://bugs.gentoo.org/761538

@zmedico zmedico changed the title asyncio: Use default sleep implementation when possible asyncio: Use default sleep / ensure_future implementations when possible Aug 31, 2024
@zmedico zmedico force-pushed the asyncio_default_sleep_impl_when_possible branch 2 times, most recently from 9836719 to 6f42f99 Compare August 31, 2024 19:10
@zmedico zmedico changed the base branch from master to prefix August 31, 2024 19:36
@zmedico zmedico changed the base branch from prefix to master August 31, 2024 19:36
When a loop argument is not given, use the default asyncio sleep
implementation and avoid unnecessary _wrap_loop usage.

Bug: https://bugs.gentoo.org/761538
Signed-off-by: Zac Medico <[email protected]>
When a loop argument is not given, use the default asyncio
ensure_future implementation and avoid unnecessary _wrap_loop
usage.

Bug: https://bugs.gentoo.org/761538
Signed-off-by: Zac Medico <[email protected]>
@zmedico zmedico force-pushed the asyncio_default_sleep_impl_when_possible branch from b1ceb7a to b30ddb1 Compare September 1, 2024 07:02
@gentoo-bot gentoo-bot merged commit b30ddb1 into gentoo:master Sep 1, 2024
13 checks passed
@zmedico zmedico deleted the asyncio_default_sleep_impl_when_possible branch September 1, 2024 07:04
This pull request was closed.
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