typo : s2/index.go by wooffie · Pull Request #1069 · klauspost/compress (original) (raw)

📝 Walkthrough

Walkthrough

The change updates the error message in the add method of the Index struct within s2/index.go. The error message now correctly compares latest.compressedOffset with compressedOffset rather than comparing latest.uncompressedOffset with uncompressedOffset. This adjustment ensures the error message accurately reflects the intended condition check.

Changes

File Change Summary
s2/index.go Updated error message in the add method to compare latest.compressedOffset with compressedOffset instead of latest.uncompressedOffset with uncompressedOffset.

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 520e81d and 309b596.

📒 Files selected for processing (1)

s2/index.go (1)

74-75: Great bug fix for the error message!

The error message now correctly shows the compressed offsets that are being compared in the condition check, instead of incorrectly referencing the uncompressed offset. This makes the error message accurately reflect the condition that triggered it, which is important for debugging.

✨ Finishing Touches


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Chat

There are 3 ways to chat with CodeRabbit:

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community