Skip to content

bpo-1635741: Port _lzma module to multiphase initialization #19382

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

Merged
corona10 merged 5 commits into from
Jun 22, 2020
Merged

bpo-1635741: Port _lzma module to multiphase initialization #19382

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
c556f77
bpo-1635741: Port _lzma module to multiphase initialization
corona10 Apr 4, 2020
14ef8fa
bpo-1635741: Fix memory leak
corona10 Jun 22, 2020
3041995
bpo-1635741: Update NEWS.d
corona10 Jun 22, 2020
60d315c
bpo-1635741: Apply PEP 7
corona10 Jun 22, 2020
f60b591
bpo-1635741: Apply Victor's review
corona10 Jun 22, 2020
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
bpo-1635741: Fix memory leak
  • Loading branch information
@corona10
corona10 committed Jun 22, 2020
commit 14ef8fa7510a2b413b070526d1f9cab5a60dac84
2 changes: 2 additions & 0 deletions Modules/_lzmamodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1563,6 +1563,7 @@ lzma_traverse(PyObject *module, visitproc visit, void *arg)
_lzma_state *state = get_lzma_state(module);
Py_VISIT(state->lzma_compressor_type);
Py_VISIT(state->lzma_decompressor_type);
Py_VISIT(state->error_type);
Py_VISIT(state->empty_tuple);
return 0;
}
Expand All @@ -1573,6 +1574,7 @@ lzma_clear(PyObject *module)
_lzma_state *state = get_lzma_state(module);
Py_CLEAR(state->lzma_compressor_type);
Py_CLEAR(state->lzma_decompressor_type);
Py_CLEAR(state->error_type);
Py_CLEAR(state->empty_tuple);
return 0;
}
Expand Down