Skip to content

gh-114258: Update AC to support @critical_section for tp_iternext - #159962

Merged
corona10 merged 6 commits into
python:mainfrom
corona10:gh-114258-clinic-next
Oct 21, 2026
Merged

corona10 merged 6 commits into
python:mainfrom
corona10:gh-114258-clinic-next

Conversation

@corona10

@corona10 corona10 commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

@corona10

Copy link
Copy Markdown
Member Author

cc @ZeroIntensity

@ZeroIntensity ZeroIntensity 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.

This looks good for tp_iternext/__next__, but I think we need to make some changes in order to support all the slots described in the issue. We could also wait to do those until we actually need them if you'd prefer, but that would mean larger diffs for those PRs (rather than just one big one now).

Comment thread Tools/clinic/libclinic/parse_args.py
Comment thread Tools/clinic/libclinic/parse_args.py

@ZeroIntensity ZeroIntensity 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.

LGTM!

@github-project-automation github-project-automation Bot moved this from Todo to In Progress in Sprint Oct 20, 2026
@corona10
corona10 enabled auto-merge (squash) October 20, 2026 22:44
@corona10
corona10 disabled auto-merge October 20, 2026 23:15
@corona10
corona10 enabled auto-merge (squash) October 20, 2026 23:16
@corona10
corona10 merged commit f9c0e1e into python:main Oct 21, 2026
66 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Sprint Oct 21, 2026
@corona10
corona10 deleted the gh-114258-clinic-next branch October 21, 2026 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sprint type-feature A feature request or enhancement

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants