[PR #444] [MERGED] Eliminate more magic constants. #1062

New Issue

claunia · 2026-01-29T20:52:20Z

claunia commented

2026-01-29 20:52:20 +00:00

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/444
Author: @eustas
Created: 10/17/2016
Status: ✅ Merged
Merged: 10/18/2016
Merged by: @eustas

Base: master ← Head: master

📝 Commits (1)

9521d96 Eliminate more magic constants.

📊 Changes

8 files changed (+24 additions, -12 deletions)

View changed files

📝 common/constants.h (+5 -0)
📝 dec/decode.c (+1 -1)
📝 enc/backward_references.c (+5 -3)
📝 enc/backward_references.h (+2 -1)
📝 enc/compress_fragment.c (+3 -2)
📝 enc/compress_fragment_two_pass.c (+3 -2)
📝 enc/encode.c (+1 -1)
📝 enc/hash.h (+4 -2)

📄 Description

Author: Ivan Nikulin

_{🔄 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/444 **Author:** [@eustas](https://github.com/eustas) **Created:** 10/17/2016 **Status:** ✅ Merged **Merged:** 10/18/2016 **Merged by:** [@eustas](https://github.com/eustas) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`9521d96`](https://github.com/google/brotli/commit/9521d968f312c14a3a30888575d14b5515f9cb40) Eliminate more magic constants. ### 📊 Changes **8 files changed** (+24 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `common/constants.h` (+5 -0) 📝 `dec/decode.c` (+1 -1) 📝 `enc/backward_references.c` (+5 -3) 📝 `enc/backward_references.h` (+2 -1) 📝 `enc/compress_fragment.c` (+3 -2) 📝 `enc/compress_fragment_two_pass.c` (+3 -2) 📝 `enc/encode.c` (+1 -1) 📝 `enc/hash.h` (+4 -2) </details> ### 📄 Description Author: Ivan Nikulin --- <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:20 +00:00

claunia referenced this issue

2026-01-29 20:54:57 +00:00

[PR #1062] [CLOSED] Move serialized dictionary feature behind the flag. #1616

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#1062