[PR #30] [MERGED] Implement a 32-bit bitstream decoder + visual studio support #607

New Issue

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

claunia commented

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

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/30
Author: @szabadka
Created: 2/25/2015
Status: ✅ Merged
Merged: 2/25/2015
Merged by: @szabadka

Base: master ← Head: master

📝 Commits (2)

6da0c5c Implement a 32-bit bitstream decoder.
f0b88cb Fixes to the encoder to support visual studio.

📊 Changes

7 files changed (+69 additions, -21 deletions)

View changed files

📝 dec/bit_reader.h (+39 -11)
📝 dec/decode.c (+11 -2)
📝 dec/streams.c (+2 -0)
📝 enc/block_splitter.cc (+10 -2)
📝 enc/fast_log.h (+2 -1)
📝 enc/port.h (+5 -0)
📝 enc/write_bits.h (+0 -5)

📄 Description

The 64-bit bitstream decoder seems to have portability problems
with emscripten/asm.js as it does not compile into working code.

_{🔄 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/30 **Author:** [@szabadka](https://github.com/szabadka) **Created:** 2/25/2015 **Status:** ✅ Merged **Merged:** 2/25/2015 **Merged by:** [@szabadka](https://github.com/szabadka) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`6da0c5c`](https://github.com/google/brotli/commit/6da0c5c9922be93e5076e5cb52efe8d2471174d2) Implement a 32-bit bitstream decoder. - [`f0b88cb`](https://github.com/google/brotli/commit/f0b88cbcdb1983bbcd680e6f0de8a21924eb025c) Fixes to the encoder to support visual studio. ### 📊 Changes **7 files changed** (+69 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `dec/bit_reader.h` (+39 -11) 📝 `dec/decode.c` (+11 -2) 📝 `dec/streams.c` (+2 -0) 📝 `enc/block_splitter.cc` (+10 -2) 📝 `enc/fast_log.h` (+2 -1) 📝 `enc/port.h` (+5 -0) 📝 `enc/write_bits.h` (+0 -5) </details> ### 📄 Description The 64-bit bitstream decoder seems to have portability problems with emscripten/asm.js as it does not compile into working code. --- <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:24 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#607