GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-146063: Add PyObject_CallFinalizerFromDealloc() to the limited C API by vstinner · Pull Request #146172 · python/cpython

Conversation

@vstinner

@vstinner vstinner commented Mar 19, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Issue: Please add PyObject_CallFinalizerFromDealloc() to stable ABI #146063

📚 Documentation preview 📚: https://cpython-previews--146172.org.readthedocs.build/

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@vstinner
pythongh-146063: Add PyObject_CallFinalizer() to the limited C API
ba44a2e 
Add PyObject_CallFinalizer() and PyObject_CallFinalizerFromDealloc()
to the limited C API.

Add a test on PyObject_CallFinalizer().
@vstinner
vstinner requested review from a team, AA-Turner and encukou as code owners March 19, 2026 14:40
@bedevere-app bedevere-app Bot mentioned this pull request Mar 19, 2026
Please add PyObject_CallFinalizerFromDealloc() to stable ABI #146063
Closed
@bedevere-app bedevere-app Bot added the awaiting core review label Mar 19, 2026
@vstinner vstinner added the topic-C-API label Mar 19, 2026
@vstinner

vstinner commented Mar 19, 2026

Copy link
Copy Markdown
Member Author

My PR lacks PyObject_CallFinalizerFromDealloc() test which is more complicated to test.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@vstinner
Remove PyObject_CallFinalizer()
2b32466
@vstinner vstinner changed the title gh-146063: Add PyObject_CallFinalizer() to the limited C API gh-146063: Add PyObject_CallFinalizerFromDealloc() to the limited C API Mar 20, 2026
@vstinner

vstinner commented Mar 20, 2026

Copy link
Copy Markdown
Member Author

I updated the PR to only add PyObject_CallFinalizerFromDealloc() to the limited C API (leave PyObject_CallFinalizer() in the non-limited C API).
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@davidhewitt

davidhewitt commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Gentle ping here - it'd be great to get this into 3.15 before beta freeze if possible! Thanks :)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@encukou

encukou commented Apr 15, 2026

Copy link
Copy Markdown
Member

WG vote: capi-workgroup/decisions#104
vstinner reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@encukou
Merge in the main branch
40fd740
@read-the-docs-community

read-the-docs-community Bot commented May 4, 2026

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32525579 | 📁 Comparing 40fd740 against main (1504bd6)

  🔍 Preview build  

4 files changed
± c-api/lifecycle.html
± c-api/stable.html
± whatsnew/3.15.html
± whatsnew/changelog.html
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
encukou
encukou approved these changes May 4, 2026
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting core review labels May 4, 2026
@vstinner
vstinner merged commit ce51c18 into python:main May 4, 2026
58 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@vstinner
vstinner deleted the call_finalizer branch May 4, 2026 13:37
@bedevere-app bedevere-app Bot removed the awaiting merge label May 4, 2026
@vstinner vstinner mentioned this pull request May 4, 2026
Add PyObject_CallFinalizerFromDealloc to Stable ABI capi-workgroup/decisions#104
Closed
3 of 5 tasks
@coderabbitai coderabbitai Bot mentioned this pull request Aug 20, 2026
cpyext: what a Cython module reaches for, and the descriptor-cache order it made visible pyre-lang/pyre#1363
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@encukou encukou encukou approved these changes
@AA-Turner AA-Turner Awaiting requested review from AA-Turner AA-Turner is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

topic-C-API

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@vstinner @davidhewitt @encukou
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.