◐ Shell
clean mode source ↗

GH-122155: Fix cases generator to correctly compute 'peek' offset for error handling by markshannon · Pull Request #122158 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@markshannon

@markshannon markshannon commented

Jul 23, 2024

edited by bedevere-app Bot

Loading

Copy link Copy Markdown

Member

Computes the "peek" offset on demand, rather than attempting to keep it up to date as the stack is modified.
This is simpler and less error prone.

I'll like to backport this to 3.13, so that the fix for #122029 can be more easily backported.

@iritkatriel

Copy link Copy Markdown

Member

There's a mypy error, but otherwise LGTM.

@markshannon markshannon merged commit 624bda7 into python:main

Jul 23, 2024

@miss-islington-app

Copy link Copy Markdown

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

@miss-islington-app

Copy link Copy Markdown

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

cherry_picker 624bda76386efd8eecf73c4ad06f997b9b25f07f 3.13

@markshannon markshannon deleted the fix-122155 branch

July 23, 2024 13:12

@markshannon markshannon restored the fix-122155 branch

July 23, 2024 13:12

@markshannon markshannon deleted the fix-122155 branch

July 23, 2024 13:12

markshannon added a commit to faster-cpython/cpython that referenced this pull request

Jul 23, 2024

@bedevere-app

bedevere-app Bot commented

Jul 23, 2024

Copy link Copy Markdown

GH-122174 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

Jul 23, 2024

nohlson pushed a commit to nohlson/cpython that referenced this pull request

Jul 24, 2024

nohlson pushed a commit to nohlson/cpython that referenced this pull request

Jul 24, 2024

markshannon added a commit that referenced this pull request

Jul 26, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iritkatriel iritkatriel iritkatriel approved these changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@markshannon @iritkatriel