chore: minor comment fix

[bradh]

Thanks @coderabbitai for reporting this at #81 (review)

[coderabbitai]

Walkthrough

A comment in the Opus::decode_body function was updated in the opus decoder module. The comment text was changed from "Find esds in mp4a or wave" to "Find d0ps in mp4a or wave". This is a documentation-only change with no modifications to the functional code logic or control flow. No public API signatures were altered.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the changeset: a minor comment fix updating 'esds' to 'd0ps' in the Opus decode function.
Description check	✅ Passed	The description is related to the changeset by referencing the upstream issue where the comment fix was identified, providing context for the change.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fef880e and cbe5090.

📒 Files selected for processing (1)

• src/moov/trak/mdia/minf/stbl/stsd/opus.rs

🧰 Additional context used

📓 Path-based instructions (3)

**/*.rs

📄 CodeRabbit inference engine (CLAUDE.md)

**/*.rs: Code must be clippy-clean (cargo clippy --all-targets --all-features -- -D warnings)
Code must be formatted with rustfmt (cargo fmt) and pass formatting checks (cargo fmt -- --check)
Write unit tests inline within modules using #[cfg(test)]

Files:

• src/moov/trak/mdia/minf/stbl/stsd/opus.rs

src/**/*.rs

📄 CodeRabbit inference engine (CLAUDE.md)

src/**/*.rs: Each MP4 atom type must implement the Atom trait with KIND, decode_body(), and encode_body()
Use Header::read_from() to handle large atoms without loading entire contents into memory
All operations should return Result using custom error types defined in error.rs
Use the custom Buf/BufMut traits for contiguous byte slices instead of ad-hoc buffer handling

Files:

• src/moov/trak/mdia/minf/stbl/stsd/opus.rs

src/moov/**

📄 CodeRabbit inference engine (CLAUDE.md)

Place movie metadata atom implementations under src/moov/ (and its hierarchy: trak/mdia/minf/stbl/stsd)

Files:

• src/moov/trak/mdia/minf/stbl/stsd/opus.rs

🧠 Learnings (1)

📚 Learning: 2025-08-31T04:06:57.364Z

Learnt from: CR
Repo: kixelated/mp4-atom PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-31T04:06:57.364Z
Learning: Applies to src/**/*.rs : Each MP4 atom type must implement the Atom trait with KIND, decode_body(), and encode_body()

Applied to files:

• src/moov/trak/mdia/minf/stbl/stsd/opus.rs

🔇 Additional comments (1)

src/moov/trak/mdia/minf/stbl/stsd/opus.rs (1)

18-18: LGTM!

Good catch fixing this comment. The code is indeed searching for Dops atoms (the Opus-specific configuration box with FourCC dOps), not esds (which is for AAC). The comment now accurately reflects the decoder's behavior.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}