Skip to content

fix: minizip's iowin32.h/c aren't compiled/installed with MinGW#12

Closed
jk3064 wants to merge 1 commit intomadler:masterfrom
jk3064:master
Closed

fix: minizip's iowin32.h/c aren't compiled/installed with MinGW#12
jk3064 wants to merge 1 commit intomadler:masterfrom
jk3064:master

Conversation

@jk3064
Copy link
Copy Markdown
Contributor

@jk3064 jk3064 commented Mar 26, 2012

No description provided.

@jk3064 jk3064 closed this Mar 26, 2012
geoff-nixon referenced this pull request in geoff-nixon/zlib May 6, 2016
ryantrinkle pushed a commit to obsidiansystems/zlib that referenced this pull request Jun 7, 2020
Change the window bits range from 8..15 to 9..15
blitzy bot pushed a commit to Blitzy-Sandbox/blitzy-zlib that referenced this pull request Feb 25, 2026
…ount/buffer continuity, detect_data_type propagation, constant dedup, gz/read workaround removal

Resolved all 12 findings from Checkpoint 3 review:

MAJOR fixes:
- #1: detect_data_type() result now stored in DeflateState.data_type and
  propagated to stream.data_type via flush_block_only (trees.rs, state.rs, algorithm.rs)
- madler#6: BufError logic flattened to match C inflate.c L1150 exactly (inflate/mod.rs)
- madler#7: Check mode double-counting fixed with out_start=left reset matching C L1081
- madler#8: Removed avail_out==0 StreamError guard to allow header-only processing
- madler#9: inflate_fast now shares output_buf with slow path (start=0), fixing
  back-reference continuity across slow-to-fast transitions

MINOR fixes:
- #2: Added debug_assert to put_short/put_byte for buffer overflow detection
- madler#3: Removed unused TOO_FAR from hash.rs
- madler#4: Consolidated MIN_LOOKAHEAD to deflate/mod.rs, imported in submodules
- madler#10: Documented borrow-checker input copy requirement in inflate
- madler#11: inflate_get_header takes Box<GzHeader>; added header()/header_mut() accessors

INFO fixes:
- madler#5: Documented allocation strategy in flush_block_only; added capacity hints
- madler#12: Removed INFLATE_SAFE_OUT=257 workaround in gz/read.rs

FFI layer updated to Box::new() header in inflateGetHeader wrapper.
All 269 tests pass, zero new compilation errors, zero new clippy violations.
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.

1 participant