[PR #733] [CLOSED] Improved BrotliParseAsUTF8 function #1370

New Issue

claunia · 2026-01-29T20:53:45Z

claunia commented

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

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/733
Author: @PopeSpaceous
Created: 12/11/2018
Status: ❌ Closed

Base: master ← Head: UTF8Improvement

📝 Commits (1)

db54ba8 Improved BrotliParseAsUTF8 function

📊 Changes

1 file changed (+7 additions, -28 deletions)

View changed files

📝 c/enc/utf8_util.c (+7 -28)

📄 Description

Hi, I was working on optimizing a compression software and was looking into brotli.
Using some benchmarking tools I found that there was room for improvement in the BrotliParseAsUTF8 function.
I have made some changes to the code and when you run files around a GB large you gain about a 20% improvement in speed, with no changes to the file size or changes to the file themselves.
I have outlined more of my testing, benchmarking and code on my site if you wanted to use those as a reference for my changes:
Benchmarking
Implimentation

_{🔄 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/733 **Author:** [@PopeSpaceous](https://github.com/PopeSpaceous) **Created:** 12/11/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `UTF8Improvement` --- ### 📝 Commits (1) - [`db54ba8`](https://github.com/google/brotli/commit/db54ba8910434fe4bf2b4c463be8e66331e76cf4) Improved BrotliParseAsUTF8 function ### 📊 Changes **1 file changed** (+7 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `c/enc/utf8_util.c` (+7 -28) </details> ### 📄 Description Hi, I was working on optimizing a compression software and was looking into brotli. Using some benchmarking tools I found that there was room for improvement in the BrotliParseAsUTF8 function. I have made some changes to the code and when you run files around a GB large you gain about a 20% improvement in speed, with no changes to the file size or changes to the file themselves. I have outlined more of my testing, benchmarking and code on my site if you wanted to use those as a reference for my changes: [Benchmarking](http://compileofcrap.com/SPO600/8.html) [Implimentation](http://compileofcrap.com/SPO600/9.html) --- <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:53:45 +00:00

claunia closed this issue

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

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#1370