[PR #112] [MERGED] Speedups and fixes to the decoder. #720

New Issue

claunia · 2026-01-29T20:50:31Z

claunia commented

2026-01-29 20:50:31 +00:00

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/112
Author: @szabadka
Created: 6/12/2015
Status: ✅ Merged
Merged: 6/12/2015
Merged by: @szabadka

Base: master ← Head: master

📝 Commits (1)

641bc15 Speedups and fixes to the decoder.

📊 Changes

3 files changed (+18 additions, -14 deletions)

View changed files

📝 dec/bit_reader.h (+8 -8)
📝 dec/decode.c (+4 -1)
📝 dec/huffman.c (+6 -5)

📄 Description

Read data by 4-byte runs.
This resolves unaligned read (Bus error) on arm-android.
Get rid of malloc/free in BrotliBuildHuffmanTable.
Tweak order of instructions when reading Huffman codes.

_{🔄 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/112 **Author:** [@szabadka](https://github.com/szabadka) **Created:** 6/12/2015 **Status:** ✅ Merged **Merged:** 6/12/2015 **Merged by:** [@szabadka](https://github.com/szabadka) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`641bc15`](https://github.com/google/brotli/commit/641bc158826f86089e6e49ba755866d836dae13a) Speedups and fixes to the decoder. ### 📊 Changes **3 files changed** (+18 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `dec/bit_reader.h` (+8 -8) 📝 `dec/decode.c` (+4 -1) 📝 `dec/huffman.c` (+6 -5) </details> ### 📄 Description - Read data by 4-byte runs. This resolves unaligned read (Bus error) on arm-android. - Get rid of malloc/free in BrotliBuildHuffmanTable. - Tweak order of instructions when reading Huffman codes. --- <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:50:31 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#720