[PR #270] [MERGED] Fix sign-comparison warnings #876

New Issue

claunia · 2026-01-29T20:51:23Z

claunia commented

2026-01-29 20:51:23 +00:00

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/270
Author: @eustas
Created: 11/23/2015
Status: ✅ Merged
Merged: 11/23/2015
Merged by: @eustas

Base: master ← Head: master

📝 Commits (1)

bb26d19 Fix sign-comparison warnings

📊 Changes

10 files changed (+30 additions, -19 deletions)

View changed files

📝 enc/block_splitter.cc (+2 -2)
📝 enc/cluster.h (+15 -10)
📝 enc/entropy_encode.cc (+2 -0)
📝 enc/hash.h (+1 -1)
📝 enc/metablock.cc (+1 -1)
📝 enc/port.h (+7 -0)
📝 enc/ringbuffer.h (+1 -3)
📝 enc/streams.cc (+0 -1)
📝 enc/streams.h (+0 -1)
📝 enc/write_bits.h (+1 -0)

📄 Description

add more debug runtime checks
minor cleanup

_{🔄 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/270 **Author:** [@eustas](https://github.com/eustas) **Created:** 11/23/2015 **Status:** ✅ Merged **Merged:** 11/23/2015 **Merged by:** [@eustas](https://github.com/eustas) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`bb26d19`](https://github.com/google/brotli/commit/bb26d1919ffb813fb1f36e3675949b172c2b51a4) Fix sign-comparison warnings ### 📊 Changes **10 files changed** (+30 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `enc/block_splitter.cc` (+2 -2) 📝 `enc/cluster.h` (+15 -10) 📝 `enc/entropy_encode.cc` (+2 -0) 📝 `enc/hash.h` (+1 -1) 📝 `enc/metablock.cc` (+1 -1) 📝 `enc/port.h` (+7 -0) 📝 `enc/ringbuffer.h` (+1 -3) 📝 `enc/streams.cc` (+0 -1) 📝 `enc/streams.h` (+0 -1) 📝 `enc/write_bits.h` (+1 -0) </details> ### 📄 Description - add more debug runtime checks - minor cleanup --- <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:51:23 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#876