[PR #433] [MERGED] Update encoder #1046

Open
opened 2026-01-29 20:52:16 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/433
Author: @eustas
Created: 9/21/2016
Status: Merged
Merged: 9/21/2016
Merged by: @eustas

Base: masterHead: master


📝 Commits (1)

📊 Changes

17 files changed (+356 additions, -172 deletions)

View changed files

📝 dec/port.h (+1 -1)
📝 enc/brotli_bit_stream.c (+17 -9)
📝 enc/command.h (+14 -6)
📝 enc/compress_fragment.c (+50 -15)
📝 enc/compress_fragment.h (+3 -1)
📝 enc/compress_fragment_two_pass.c (+52 -20)
📝 enc/compress_fragment_two_pass.h (+3 -1)
📝 enc/context.h (+1 -1)
📝 enc/encode.c (+190 -81)
📝 enc/entropy_encode_static.h (+1 -1)
📝 enc/fast_log.h (+1 -1)
📝 enc/hash.h (+1 -1)
📝 enc/metablock.c (+2 -1)
📝 enc/port.h (+1 -10)
📝 enc/prefix.h (+3 -1)
📝 include/brotli/encode.h (+16 -22)
📝 include/brotli/port.h (+0 -0)

📄 Description

  • moved common/port.h to includes/port.h
  • replaced more magic numbers with constants
  • artificially limited window size to 2^18 for quality 0 and 1
  • used fixed shifts for quality 0 and 1 hashes
  • removed BrotliEncoderWriteMetadata
  • added BROTLI_OPERATION_EMIT_METADATA instead
  • deprecated low-level API
  • fixed MSVC warnings

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/google/brotli/pull/433 **Author:** [@eustas](https://github.com/eustas) **Created:** 9/21/2016 **Status:** ✅ Merged **Merged:** 9/21/2016 **Merged by:** [@eustas](https://github.com/eustas) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`0a63f99`](https://github.com/google/brotli/commit/0a63f99db9eeb8a15dc3941b96802141b1993912) Update encoder ### 📊 Changes **17 files changed** (+356 additions, -172 deletions) <details> <summary>View changed files</summary> 📝 `dec/port.h` (+1 -1) 📝 `enc/brotli_bit_stream.c` (+17 -9) 📝 `enc/command.h` (+14 -6) 📝 `enc/compress_fragment.c` (+50 -15) 📝 `enc/compress_fragment.h` (+3 -1) 📝 `enc/compress_fragment_two_pass.c` (+52 -20) 📝 `enc/compress_fragment_two_pass.h` (+3 -1) 📝 `enc/context.h` (+1 -1) 📝 `enc/encode.c` (+190 -81) 📝 `enc/entropy_encode_static.h` (+1 -1) 📝 `enc/fast_log.h` (+1 -1) 📝 `enc/hash.h` (+1 -1) 📝 `enc/metablock.c` (+2 -1) 📝 `enc/port.h` (+1 -10) 📝 `enc/prefix.h` (+3 -1) 📝 `include/brotli/encode.h` (+16 -22) 📝 `include/brotli/port.h` (+0 -0) </details> ### 📄 Description - moved `common/port.h` to `includes/port.h` - replaced more magic numbers with constants - artificially limited window size to 2^18 for quality 0 and 1 - used fixed shifts for quality 0 and 1 hashes - removed `BrotliEncoderWriteMetadata` - added `BROTLI_OPERATION_EMIT_METADATA` instead - deprecated low-level API - fixed MSVC warnings --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 20:52:16 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/brotli#1046