FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.13] gh-112346: Always set OS byte to 255, simpler gzip.compress function. (GH-120486) by miss-islington · Pull Request #120563 · python/cpython · GitHub

/ cpython Public

[3.13] gh-112346: Always set OS byte to 255, simpler gzip.compress function. (GH-120486) - #120563

Merged
gpshead merged 1 commit into
python:3.13from
miss-islington:backport-08d09cf-3.13
Jun 15, 2024
Merged

[3.13] gh-112346: Always set OS byte to 255, simpler gzip.compress function. (GH-120486)#120563
gpshead merged 1 commit into
python:3.13from
miss-islington:backport-08d09cf-3.13

Conversation

miss-islington commented Jun 15, 2024
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

This matches the output behavior in 3.10 and earlier; the optimization in 3.11 allowed the zlib library's "os" value to be filled in instead in the circumstance when mtime was 0. this keeps things consistent.
(cherry picked from commit 08d09cf)

Co-authored-by: Ruben Vorderman r.h.p.vorderman@lumc.nl


📚 Documentation preview 📚: https://cpython-previews--120563.org.readthedocs.build/

…ction. (pythonGH-120486)

This matches the output behavior in 3.10 and earlier; the optimization in 3.11 allowed the zlib library's "os" value to be filled in instead in the circumstance when mtime was 0.  this keeps things consistent.
(cherry picked from commit 08d09cf)

Co-authored-by: Ruben Vorderman <r.h.p.vorderman@lumc.nl>
gpshead enabled auto-merge (squash) June 15, 2024 18:55
gpshead merged commit a19bb26 into python:3.13 Jun 15, 2024
miss-islington deleted the backport-08d09cf-3.13 branch January 2, 2026 16:57
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL