scottlamb / retina

High-level RTSP multimedia streaming library, in Rust
https://crates.io/crates/retina
Apache License 2.0
220 stars 46 forks source link

AAC depacketization unit tests #48

Closed scottlamb closed 2 years ago

scottlamb commented 2 years ago

Right now the loss_since_mark fields in aac.rs talk about how they're important for determining if a fragment is complete or not:

https://github.com/scottlamb/retina/blob/0178e7fbc20453fdb49894aa669f9af089285a45/src/codec/aac.rs#L437-L441

...but they're not actually used or passed on in the AudioFrame. That's suspect. But there are no tests, and I want to add tests before messing with the logic to know I'm making things better, not worse.