[PR #189] [MERGED] Move literal cost computation to where it's used. #789

New Issue

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

claunia commented

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

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/189
Author: @szabadka
Created: 10/1/2015
Status: ✅ Merged
Merged: 10/1/2015
Merged by: @szabadka

Base: master ← Head: master

📝 Commits (1)

4c37566 Move literal cost computation to where it's used.

📊 Changes

11 files changed (+155 additions, -199 deletions)

View changed files

📝 enc/Makefile (+1 -1)
📝 enc/backward_references.cc (+9 -14)
📝 enc/backward_references.h (+0 -2)
📝 enc/encode.cc (+6 -90)
📝 enc/encode.h (+0 -3)
📝 enc/encode_parallel.cc (+3 -76)
📝 enc/literal_cost.cc (+9 -6)
📝 enc/literal_cost.h (+2 -7)
➕ enc/utf8_util.cc (+90 -0)
➕ enc/utf8_util.h (+33 -0)
📝 setup.py (+2 -0)

📄 Description

Move utf8 heuristics functions to their own file.

_{🔄 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/189 **Author:** [@szabadka](https://github.com/szabadka) **Created:** 10/1/2015 **Status:** ✅ Merged **Merged:** 10/1/2015 **Merged by:** [@szabadka](https://github.com/szabadka) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`4c37566`](https://github.com/google/brotli/commit/4c37566f4bf06df1b592daedf81025e4898fc719) Move literal cost computation to where it's used. ### 📊 Changes **11 files changed** (+155 additions, -199 deletions) <details> <summary>View changed files</summary> 📝 `enc/Makefile` (+1 -1) 📝 `enc/backward_references.cc` (+9 -14) 📝 `enc/backward_references.h` (+0 -2) 📝 `enc/encode.cc` (+6 -90) 📝 `enc/encode.h` (+0 -3) 📝 `enc/encode_parallel.cc` (+3 -76) 📝 `enc/literal_cost.cc` (+9 -6) 📝 `enc/literal_cost.h` (+2 -7) ➕ `enc/utf8_util.cc` (+90 -0) ➕ `enc/utf8_util.h` (+33 -0) 📝 `setup.py` (+2 -0) </details> ### 📄 Description Move utf8 heuristics functions to their own file. --- <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:55 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#789