[PR #123] [MERGED] Fix some VS compilation errors in the encoder. #734

New Issue

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

claunia commented

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

📋 Pull Request Information

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

Base: master ← Head: master

📝 Commits (1)

95ddb48 Fix some VS compilation errors in the encoder.

📊 Changes

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

View changed files

📝 enc/backward_references.cc (+7 -4)

📄 Description

Use std::numeric_limits::infinity() instead of 1.0 / 0.0
Use FastLog2() instead of log2() in cost model

_{🔄 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/123 **Author:** [@szabadka](https://github.com/szabadka) **Created:** 6/29/2015 **Status:** ✅ Merged **Merged:** 6/29/2015 **Merged by:** [@szabadka](https://github.com/szabadka) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`95ddb48`](https://github.com/google/brotli/commit/95ddb48a116735670e50cfdd8f23a9a505cd250d) Fix some VS compilation errors in the encoder. ### 📊 Changes **1 file changed** (+7 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `enc/backward_references.cc` (+7 -4) </details> ### 📄 Description - Use std::numeric_limits<double>::infinity() instead of 1.0 / 0.0 - Use FastLog2() instead of log2() in cost model --- <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:36 +00:00

claunia closed this issue

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

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#734