10BC0 gh-132983: Clean-ups for `_zstd` by AA-Turner · Pull Request #133670 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-132983: Clean-ups for _zstd #133670

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
merged 16 commits into from
May 9, 2025
Merged

gh-132983: Clean-ups for _zstd #133670

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
0c3c667
zstd: replace star import with explicit names
AA-Turner May 8, 2025
e02f66c
zstd: factor common parts of set_zstd_error()
AA-Turner May 8, 2025
686ae61
zstd: Remove ZSTD_versionString from error messages
AA-Turner May 8, 2025
7ea1ac1
zstd: Simplify version_info setup
AA-Turner May 8, 2025
9774338
zstd: Replace _compressionLevel_values with ZSTD_CLEVEL_DEFAULT
AA-Turner May 8, 2025
d1dea98
zstd: Remove _ZSTD_CStreamSizes, replace _ZSTD_DStreamSizes with ZSTD…
AA-Turner May 8, 2025
ac1e39c
zstd: Remove _ZSTD_CONFIG
AA-Turner May 8, 2025
1a54bf3
zstd: Replace add_parameters and ADD_INT_PREFIX_MACRO with PyModule_A…
AA-Turner May 8, 2025
9cb7a3d
Merge branch 'main' into zstd-clean
AA-Turner May 8, 2025
663e666
Revert "zstd: factor common parts of set_zstd_error()"
AA-Turner May 8, 2025
dd1380d
Use PyModule_Add
AA-Turner May 9, 2025
fcd6561
Check failure cases for int macros
AA-Turner May 9, 2025
06a4723
Remove ERR_SET_PLEDGED_INPUT_SIZE
AA-Turner May 9, 2025
ef2634d
Use PyLong_FromSize_t
AA-Turner May 9, 2025
ff28bf2
Use runtime version
AA-Turner May 9, 2025
3d3dbf7
Calculate zstd_version_info in Python
AA-Turner May 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
Use PyModule_Add
  • Loading branch information
@AA-Turner
AA-Turner committed May 9, 2025
commit dd1380dfa6264eb918af222e48e5c5d83165bcaf
8 changes: 3 additions & 5 deletions Modules/_zstd/_zstdmodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -568,13 +568,11 @@ add_vars_to_module(PyObject *module)
}

/* zstd_version_info, tuple of (int, int, int) */
PyObject *vi = Py_BuildValue("BBB", ZSTD_VERSION_MAJOR, ZSTD_VERSION_MINOR,
ZSTD_VERSION_RELEASE);
if (PyModule_AddObjectRef(module, "zstd_version_info", vi) < 0) {
Py_XDECREF(vi);
if (PyModule_Add(module, "zstd_version_info",
Py_BuildValue("BBB", ZSTD_VERSION_MAJOR, ZSTD_VERSION_MINOR,
ZSTD_VERSION_RELEASE)) < 0) {
return -1;
}
Py_DECREF(vi);

/* ZSTD_CLEVEL_DEFAULT, int */
#if ZSTD_VERSION_NUMBER >= 10500
Expand Down
0