Skip to content

gh-103793: Defer formatting task name #103767

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.

Sign up for GitHub

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

Merged
gvanrossum merged 14 commits into from
Apr 29, 2023
Merged

gh-103793: Defer formatting task name #103767

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
e8353cc
Defer formatting task name
itamaro Apr 24, 2023
1822ff8
Defer task name formatting also in Python Task impl
itamaro Apr 24, 2023
74d0084
Add NEWS and whatsnew entries
itamaro Apr 24, 2023
dbc7047
Merge branch 'main' into defer-task-name-formatting
ambv Apr 25, 2023
2bb6c1e
Merge branch 'main' into defer-task-name-formatting
itamaro Apr 25, 2023
2efd376
replace storing the task counter on task construction with just getti…
itamaro Apr 25, 2023
6cb321b
Merge branch 'defer-task-name-formatting' of github.com:itamaro/cpyth…
itamaro Apr 25, 2023
7cde290
remove redundant return
itamaro Apr 25, 2023
4b50646
Merge branch 'main' into defer-task-name-formatting
itamaro Apr 25, 2023
a155190
revert changes to python version
itamaro Apr 25, 2023
fb2bd7f
pass error up if PyUnicode_FromFormat returns NULL
itamaro Apr 25, 2023
41e9a59
Switch back to original "assign task number at construction" but stor…
itamaro Apr 26, 2023
712edf9
Add test that using an explicit PyLong as task name does not accident…
itamaro Apr 26, 2023
592d44b
Merge branch 'main' into defer-task-name-formatting
itamaro Apr 26, 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
Prev Previous commit
Next Next commit
revert changes to python version
  • Loading branch information
@itamaro
itamaro committed Apr 25, 2023
commit a155190e96b1666683c45cc716915d52f32a1e25
5 changes: 1 addition & 4 deletions Lib/asyncio/tasks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,7 @@ def __init__(self, coro, *, loop=None, name=None, context=None):
raise TypeError(f"a coroutine was expected, got {coro!r}")

if name is None:
# optimization: defer task name formatting to first get_name
self._name = None
self._name = f'Task-{_task_name_counter()}'
else:
self._name = str(name)

Expand Down Expand Up @@ -144,8 +143,6 @@ def get_context(self):
return self._context

def get_name(self):
if self._name is None:
self._name = f'Task-{_task_name_counter()}'
return self._name

def set_name(self, value):
Expand Down