Resolve remaining `clippy::identity_op` warnings
Silence `clippy::needless_update` warning in tests
Remove needless use of `vec![…]` (vs. `[…]`)
Remove redundant `t as T` cast
Replace `assert!(true)` with `{}`
Replace `assert!(false, …)`/`assert!(false)` with `panic!(…)`/`panic!()`
Unify `assert_eq!(…)`/`assert_ne!(…)` to take args in order of `…!(actual, expected)`While there isn't an explicit convention for this in Rust the stdlib docs all use `assert_eq!(actual, expected)`
Unify `assert_eq!(…)`/`assert_ne!(…)` to take args in order of `…!(actual, expected)`While there isn't an explicit convention for this in Rust the stdlib docs all use `assert_eq!(actual, expected)`, making it a sort of implicit convention.
show more ...
Don't generate empty TWCC feedback packets (#324)* Don't generate empty TWCC feedback packets The previous version of this code would very often generate a feedback packet with no chunks. These
Don't generate empty TWCC feedback packets (#324)* Don't generate empty TWCC feedback packets The previous version of this code would very often generate a feedback packet with no chunks. These packets would be ignored by libWebRTC which logs an error message saying: ``` Buffer too small (16 bytes) to fit a FeedbackPacket. Minimum size = 18 ``` When receiving such a packet. It also felt weird to clear the received packets when no chunks were generated for those packets.
Use MissedTickBehavior::Skip for TWCC ticker (#323)
Disable some flaky tests on windows
Move all to top level