Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-110771: Decompose run_forever() into parts#110773

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
gvanrossum merged 12 commits intopython:mainfromfreakboy3742:decomposed-event-loop
Oct 13, 2023
Merged
Show file tree
Hide file tree
Changes from1 commit
Commits
Show all changes
12 commits
Select commitHold shift + click to select a range
a0c1fde
Decompose run_forever() into parts to allow external usage.
freakboy3742Oct 12, 2023
67bfc8f
📜🤖 Added by blurb_it.
blurb-it[bot]Oct 13, 2023
3dcf97b
Add docs for run_forever_setup and run_forever_cleanup.
freakboy3742Oct 13, 2023
35271dc
Ensure failures during setup don't prevent cleanup.
freakboy3742Oct 13, 2023
6672f68
Add tests for custom event loop implementation.
freakboy3742Oct 13, 2023
54349c3
Store pre-event loop state as a protected variable.
freakboy3742Oct 13, 2023
e6226a5
Add a missing super() call.
freakboy3742Oct 13, 2023
23852d0
Improvements to docs, including a prototype custom event loop.
freakboy3742Oct 13, 2023
e7f892e
Correct prototype of run_forever_cleanup on Windows Proactor.
freakboy3742Oct 13, 2023
f944944
Merge branch 'main' into decomposed-event-loop
freakboy3742Oct 13, 2023
3143126
Keep the setup/cleanup methods as protected API.
freakboy3742Oct 13, 2023
1b3e3d8
Merge branch 'main' into decomposed-event-loop
freakboy3742Oct 13, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
PrevPrevious commit
NextNext commit
Store pre-event loop state as a protected variable.
  • Loading branch information
@freakboy3742
freakboy3742 committedOct 13, 2023
commit54349c3decf269aac4abc75bc0459ec7283c323f
44 changes: 18 additions & 26 deletionsLib/asyncio/base_events.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -400,6 +400,8 @@ def __init__(self):
self._clock_resolution = time.get_clock_info('monotonic').resolution
self._exception_handler = None
self.set_debug(coroutines._is_debug_mode())
# The preserved state of async generator hooks.
self._old_agen_hooks = None
# In debug mode, if the execution of a callback or a step of a task
# exceed this duration in seconds, the slow callback/task is logged.
self.slow_callback_duration = 0.1
Expand DownExpand Up@@ -602,21 +604,17 @@ def _check_running(self):
'Cannot run the event loop while another loop is running')

def run_forever_setup(self):
"""Set up an event loop so that it is ready to start actively looping
to process events.
"""Prepare the run loop to process events.

Returns the state that must be restored when the loop concludes. This state
should be passed in as arguments to ``run_forever_cleanup()``.

This method is only needed if you are writing your own event loop, with
customized ``run_forever`` semantics (e.g., integrating a GUI event loop
with Python's event loop).
This method should be used as part of the ``run_forever()``
implementation in a custom event loop subclass (e.g., integrating a GUI
event loop with Python's event loop).
"""
self._check_closed()
self._check_running()
self._set_coroutine_origin_tracking(self._debug)

old_agen_hooks = sys.get_asyncgen_hooks()
self._old_agen_hooks = sys.get_asyncgen_hooks()
self._thread_id = threading.get_ident()
sys.set_asyncgen_hooks(
firstiter=self._asyncgen_firstiter_hook,
Expand All@@ -625,38 +623,32 @@ def run_forever_setup(self):

events._set_running_loop(self)

return (old_agen_hooks,)

def run_forever_cleanup(self, original_state):
"""Clean up an event loop after the event loop finishes the looping over
events.

Restores any state preserved by ``run_forever_setup()``.
def run_forever_cleanup(self):
"""Clean up after an event loop finishes the looping over events.

This methodis only needed if you are writing your own event loop, with
customized ``run_forever`` semantics(e.g., integrating a GUI event loop
with Python's event loop).
This methodshould be used as part of the ``run_forever()``
implementation in a custom event loop subclass(e.g., integrating a GUI
event loopwith Python's event loop).
"""
self._stopping = False
self._thread_id = None
events._set_running_loop(None)
self._set_coroutine_origin_tracking(False)
if original_state is not None:
old_agen_hooks, = original_state
sys.set_asyncgen_hooks(*old_agen_hooks)
# Restore any pre-existing async generator hooks.
if self._old_agen_hooks is not None:
sys.set_asyncgen_hooks(*self._old_agen_hooks)
self._old_agen_hooks = None

def run_forever(self):
"""Run until stop() is called."""
Copy link
Member

Choose a reason for hiding this comment

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

Maybe clarify in this docstring that this is what calls setup/cleanup?

# Ensure original_state has a value in case setup fails.
original_state = None
try:
original_state =self.run_forever_setup()
self.run_forever_setup()
while True:
self._run_once()
if self._stopping:
break
finally:
self.run_forever_cleanup(original_state)
self.run_forever_cleanup()

def run_until_complete(self, future):
"""Run until the Future is done.
Expand Down
4 changes: 2 additions & 2 deletionsLib/test/test_asyncio/test_base_events.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -936,7 +936,7 @@ def callback():
self.loop.call_soon(callback)

# Set up the custom event loop
orig_state =self.loop.run_forever_setup()
self.loop.run_forever_setup()

# Confirm the loop has been started
self.assertEqual(asyncio.get_running_loop(), self.loop)
Expand All@@ -947,7 +947,7 @@ def callback():
self.loop._run_once()

# Clean up the event loop
self.loop.run_forever_cleanup(orig_state)
self.loop.run_forever_cleanup()

# Confirm the loop has been cleaned up
with self.assertRaises(RuntimeError):
Expand Down

[8]ページ先頭

©2009-2025 Movatter.jp