8000 gh-129640: Add tests for reference cycle in `gzip.GzipFile` by Mr-Sunglasses · Pull Request #130916 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-129640: Add tests for reference cycle in gzip.GzipFile #130916

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.

Sign up for GitHub

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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

gh-129640: Add tests for reference cycle in gzip.GzipFile #130916

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
06999db
gh-129640: Fix reference cycle in `gzip.GzipFile`
Mr-Sunglasses Mar 6, 2025
79b7f6c
test: Add test for reference cycle in `gzip.GzipFile`
Mr-Sunglasses Mar 6, 2025
327f877
chore: add news entry
Mr-Sunglasses Mar 6, 2025
eefc287
docs: Add refence to the function in the NEWS
Mr-Sunglasses Mar 6, 2025
ee93a71
test: Add refence to the issue in the test comment.
Mr-Sunglasses Mar 6, 2025
c17135e
Merge branch 'main' into fix/129640
Mr-Sunglasses Mar 6, 2025
0dbcd08
Merge branch 'main' into fix/129640
Mr-Sunglasses Mar 7, 2025
5b9f6e6
REVERT the changes in `gzip` module
Mr-Sunglasses Mar 8, 2025
8826606
Merge branch 'fix/129640' of github.com:Mr-Sunglasses/cpython into fi…
Mr-Sunglasses Mar 8, 2025
67b4dd2
chore: removed news entry
Mr-Sunglasses Mar 8, 2025
e6aa09d
docs: Add the reference of `GH-129726` in tests
Mr-Sunglasses Mar 8, 2025
94fa2a8
Merge branch 'main' into fix/129640
Mr-Sunglasses Mar 8, 2025
13f1006
Merge branch 'main' into fix/129640
Mr-Sunglasses Mar 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
REVERT the changes in gzip module
  • Loading branch information
@Mr-Sunglasses
Mr-Sunglasses committed Mar 8, 2025
commit 5b9f6e6904f79894179d275fa587cc667b145237
3 changes: 0 additions & 3 deletions Lib/gzip.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,9 +143,6 @@ def seekable(self):
def writable(self):
return True

def __del__(self):
del self.gzip_file


class GzipFile(_compression.BaseStream):
"""The GzipFile class simulates most of the methods of a file object with
Expand Down
0