Skip to content

Conversation

@AZero13
Copy link
Contributor

@AZero13 AZero13 commented Dec 8, 2025

Move deref of laststring to below the error checking so the deref is applied after the object in strings is replaced.

@AZero13 AZero13 requested a review from lysnikolaou as a code owner December 8, 2025 05:01
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@eendebakpt
Copy link
Contributor

@AZero13 Can you write a news entry and create a corresponding issue? See https://devguide.python.org/getting-started/pull-request-lifecycle/ for details

@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@AZero13 AZero13 changed the title Remove erroneous dereference of laststring gh-142433: Move deref to below the error when checking for laststring. Dec 8, 2025
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@AZero13 AZero13 requested a review from eendebakpt December 8, 2025 22:35
@AZero13 AZero13 changed the title gh-142433: Move deref to below the error when checking for laststring. gh-142433: Move deref to below the error when checking for laststring Dec 8, 2025
…string

Move deref of laststring to below the error checking so the deref is applied after the object in strings is replaced.
@eendebakpt
Copy link
Contributor

@AZero13 please avoid force pushes, it makes reviewing harder

Copy link
Contributor

@eendebakpt eendebakpt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The news entry needs a bit of work, but the change to the code looks good.

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vstinner vstinner added the needs backport to 3.14 bugs and security fixes label Dec 9, 2025
@vstinner vstinner merged commit 785268f into python:main Dec 10, 2025
84 of 85 checks passed
@miss-islington-app
Copy link

Thanks @AZero13 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 10, 2025
…string (pythonGH-142402)

Move deref of laststring to below the error checking so the deref
is applied after the object in strings is replaced.
(cherry picked from commit 785268f)

Co-authored-by: AZero13 <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Dec 10, 2025

GH-142524 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Dec 10, 2025
@vstinner
Copy link
Member

Merged, thanks.

@AZero13 AZero13 deleted the deref-template branch December 10, 2025 15:45
vstinner pushed a commit that referenced this pull request Dec 10, 2025
…tstring (GH-142402) (#142524)

gh-142433: Move deref to below the error when checking for laststring (GH-142402)

Move deref of laststring to below the error checking so the deref
is applied after the object in strings is replaced.
(cherry picked from commit 785268f)

Co-authored-by: AZero13 <[email protected]>
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.

4 participants