Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-128991: Release the enter frame reference within bdb callback #128992

Merged
merged 2 commits into from
Jan 18, 2025

Conversation

gaogaotiantian
Copy link
Member

@gaogaotiantian gaogaotiantian commented Jan 18, 2025

Use a context manager to release the enter frame reference, instead of doing it in continue.

@gaogaotiantian gaogaotiantian added needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels Jan 18, 2025
@gaogaotiantian
Copy link
Member Author

In theory, this is not a bug fix and should not be backported. However, this is required for #125549. Also it would be nice if we don't hold the frame reference for too long in previous versions.

@gaogaotiantian gaogaotiantian merged commit 61b35f7 into python:main Jan 18, 2025
47 checks passed
@miss-islington-app
Copy link

Thanks @gaogaotiantian for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

@gaogaotiantian gaogaotiantian deleted the refactor-enterframe branch January 18, 2025 21:53
@miss-islington-app
Copy link

Sorry, @gaogaotiantian, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 61b35f74aa4a6ac606635e245147ff3658628d99 3.13

@miss-islington-app
Copy link

Sorry, @gaogaotiantian, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 61b35f74aa4a6ac606635e245147ff3658628d99 3.12

@bedevere-app
Copy link

bedevere-app bot commented Jan 18, 2025

GH-129002 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jan 18, 2025
gaogaotiantian added a commit to gaogaotiantian/cpython that referenced this pull request Jan 18, 2025
…callback (pythonGH-128992)

* Release the enter frame reference within bdb callback

* 📜🤖 Added by blurb_it.

---------

(cherry picked from commit 61b35f7)

Co-authored-by: Tian Gao <[email protected]>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
@bedevere-app
Copy link

bedevere-app bot commented Jan 18, 2025

GH-129003 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Jan 18, 2025
gaogaotiantian added a commit that referenced this pull request Jan 18, 2025
#129002)

[3.13] gh-128991: Release the enter frame reference within bdb callback (GH-128992)

* Release the enter frame reference within bdb callback

* 📜🤖 Added by blurb_it.

---------

(cherry picked from commit 61b35f7)
gaogaotiantian added a commit that referenced this pull request Jan 18, 2025
#129003)

[3.12] gh-128991: Release the enter frame reference within bdb callback (GH-128992)

* Release the enter frame reference within bdb callback

* 📜🤖 Added by blurb_it.

---------

(cherry picked from commit 61b35f7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants