[PR #12] [MERGED] Decoder code cleanup. #587

New Issue

claunia · 2026-01-29T20:46:17Z

claunia commented

2026-01-29 20:46:17 +00:00

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/12
Author: @szabadka
Created: 10/28/2014
Status: ✅ Merged
Merged: 10/28/2014
Merged by: @szabadka

Base: master ← Head: master

📝 Commits (1)

460dda1 Decoder code cleanup.

📊 Changes

3 files changed (+9 additions, -9 deletions)

View changed files

📝 dec/decode.c (+3 -3)
📝 dec/safe_malloc.h (+3 -3)
📝 dec/transform.h (+3 -3)

📄 Description

Use a tighter upper bound on the Huffman table size.
Remove unused argument len in ToUpperCase.
Remove unused no-op assignment in BrotliDecompress.
Fix include guard name.

_{🔄 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/12 **Author:** [@szabadka](https://github.com/szabadka) **Created:** 10/28/2014 **Status:** ✅ Merged **Merged:** 10/28/2014 **Merged by:** [@szabadka](https://github.com/szabadka) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`460dda1`](https://github.com/google/brotli/commit/460dda1b502485775a0f89e6b99ec753ec3f0998) Decoder code cleanup. ### 📊 Changes **3 files changed** (+9 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `dec/decode.c` (+3 -3) 📝 `dec/safe_malloc.h` (+3 -3) 📝 `dec/transform.h` (+3 -3) </details> ### 📄 Description 1) Use a tighter upper bound on the Huffman table size. 2) Remove unused argument len in ToUpperCase. 3) Remove unused no-op assignment in BrotliDecompress. 4) Fix include guard name. --- <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:46:17 +00:00

claunia closed this issue

2026-01-29 20:46:17 +00:00

claunia referenced this issue

2026-01-29 20:53:05 +00:00

[PR #587] [CLOSED] Also build test with GCC 7 and LLVM/Clang 4.0 #1227

claunia referenced this issue

2026-01-29 20:53:05 +00:00

[PR #587] Also build test with GCC 7 and LLVM/Clang 4.0 #1231

Sign in to join this conversation.