Skip to content

gh-128002: use internal llist implementation for asyncio tasks - #128256

Merged
kumaraditya303 merged 2 commits into
python:mainfrom
kumaraditya303:llist
Dec 25, 2024
Merged

gh-128002: use internal llist implementation for asyncio tasks #128256
kumaraditya303 merged 2 commits into
python:mainfrom
kumaraditya303:llist

Conversation

@kumaraditya303

@kumaraditya303kumaraditya303 commented Dec 25, 2024•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

This refactor would help further experimentation like moving to per-thread asyncio tasks lists etc.

@bedevere-appbedevere-appBot mentioned this pull request Dec 25, 2024
Closed
9 tasks
picnixz
picnixz approved these changes Dec 25, 2024
View reviewed changes

@picnixzpicnixz left a comment

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.

One nit and LGTM.

Comment threadModules/_asynciomodule.c Outdated
@kumaraditya303
kumaraditya303 merged commit 76f1785 into python:mainDec 25, 2024
@kumaraditya303
kumaraditya303 deleted the llist branch December 25, 2024 12:21
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
@kumaraditya303@srinivasreddy
pythongh-128002: use internal llist implementation for asyncio tasks (p…
f956778
…ython#128256)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@picnixzpicnixzpicnixz approved these changes
@1st11st1Awaiting requested review from 1st11st1 is a code owner
@asvetlovasvetlovAwaiting requested review from asvetlovasvetlov is a code owner
@willingcwillingcAwaiting requested review from willingcwillingc is a code owner

Assignees

No one assigned

Labels

skip newstopic-asyncio

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kumaraditya303@picnixz