admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

dcsctp: Remove handover_state from RRSendQueue

Browse Source 
It's not used as a constructor parameter, but instead invoked by calling
RestoreFromState. Removing this parameter avoids confusion.

Bug: None
Change-Id: I3bb8a0135808e3ae010be6d37439513517f71832
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/254820
Auto-Submit: Victor Boivie <boivie@webrtc.org>
Reviewed-by: Sergey Sukhanov <sergeysu@webrtc.org>
Commit-Queue: Sergey Sukhanov <sergeysu@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#36188}
This commit is contained in:
Victor Boivie 2022-03-10 20:25:10 +01:00 committed by WebRTC LUCI CQ
parent c5b9c95c88
commit 9848890590
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
net/dcsctp/tx/rr_send_queue.cc
View File

@ -32,8 +32,7 @@ RRSendQueue::RRSendQueue(absl::string_view log_prefix,
size_t buffer_size,
std::function<void(StreamID)> on_buffered_amount_low,
size_t total_buffered_amount_low_threshold,
std::function<void()> on_total_buffered_amount_low,
const DcSctpSocketHandoverState* handover_state)
std::function<void()> on_total_buffered_amount_low)
: log_prefix_(std::string(log_prefix) + "fcfs: "),
buffer_size_(buffer_size),
on_buffered_amount_low_(std::move(on_buffered_amount_low)),

3
net/dcsctp/tx/rr_send_queue.h
View File

@ -47,8 +47,7 @@ class RRSendQueue : public SendQueue {
size_t buffer_size,
std::function<void(StreamID)> on_buffered_amount_low,
size_t total_buffered_amount_low_threshold,
std::function<void()> on_total_buffered_amount_low,
const DcSctpSocketHandoverState* handover_state = nullptr);
std::function<void()> on_total_buffered_amount_low);
// Indicates if the buffer is full. Note that it's up to the caller to ensure
// that the buffer is not full prior to adding new items to it.