Skip to content

Conversation

@StanFromIreland
Copy link
Member

@StanFromIreland StanFromIreland commented Oct 22, 2025

Four bytes is four bytes :-)

Copy link
Contributor

@sergey-miryanov sergey-miryanov 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

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

The core fix looks good, but please add a test stressing this. We can remove it when we remove the u type code in 3.16.

skirpichev

This comment was marked as resolved.

@skirpichev skirpichev added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Oct 23, 2025
@skirpichev
Copy link
Contributor

This seems to be present in 3.13 and 3.14.

@ZeroIntensity ZeroIntensity merged commit aa9d0a6 into python:main Oct 23, 2025
53 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 23, 2025
(cherry picked from commit aa9d0a6)

Co-authored-by: Stan Ulbrych <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 23, 2025
(cherry picked from commit aa9d0a6)

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

bedevere-app bot commented Oct 23, 2025

GH-140498 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 Oct 23, 2025
@bedevere-app
Copy link

bedevere-app bot commented Oct 23, 2025

GH-140499 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 Oct 23, 2025
ZeroIntensity pushed a commit that referenced this pull request Oct 23, 2025
…0498)

gh-140474: Fix memory leak in `array.array` (GH-140478)
(cherry picked from commit aa9d0a6)

Co-authored-by: Stan Ulbrych <[email protected]>
ZeroIntensity pushed a commit that referenced this pull request Oct 23, 2025
…0499)

gh-140474: Fix memory leak in `array.array` (GH-140478)
(cherry picked from commit aa9d0a6)

Co-authored-by: Stan Ulbrych <[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