José Luis Millán
José Luis Millán
Update: There's actually no plan for enabling RED (currently under flag) by default in libwebrtc. Thanks @fippo for the update.
Just wondering why just not discard the packet on the transport itself. Any reason no to simply do that? ```diff diff --git a/worker/src/RTC/Transport.cpp b/worker/src/RTC/Transport.cpp index b6cf99740..52e6138b5 100644 --- a/worker/src/RTC/Transport.cpp +++...
> Just wondering why just not discard the packet on the transport itself. Any reason no to simply do that? Auto response: In order to update the transmission counters in...
> That looks better but i was worried about the gaps in the sequence numbers that discarding the packet before the Consumer could create. That's right @ggarber. We need to...
> They need to be filtered like when you filter temporal layers (no seqnum gaps). That's why I do the this->rtpSeqManager.Drop(packet->GetSequenceNumber()); that i'm assuming does that. That's correct :+1:
Please @ggarber, assign me for review after the requested changes are done.
compilation is failing
@ggarber, friendly ping.
Canceling Rust GH actions because that changes are missing.
We are completely removing any data structure for `internal`. Data structures for `data` will be reviewed afterwards.