Minor FlexFEC fixes.
BUG=webrtc:5654 Review-Url: https://codereview.webrtc.org/2720693007 Cr-Commit-Position: refs/heads/master@{#17292}
This commit is contained in:
parent
a2bb667a9b
commit
3b941bef7a
@ -30,7 +30,7 @@ namespace webrtc {
|
||||
class RtpPacketToSend;
|
||||
|
||||
// Note that this class is not thread safe, and thus requires external
|
||||
// synchronization.
|
||||
// synchronization. Currently, this is done using the lock in PayloadRouter.
|
||||
|
||||
class FlexfecSender {
|
||||
public:
|
||||
|
||||
@ -194,7 +194,7 @@ std::string VideoSendStream::Config::Rtp::ToString() const {
|
||||
if (i != flexfec.protected_media_ssrcs.size() - 1)
|
||||
ss << ", ";
|
||||
}
|
||||
ss << ']';
|
||||
ss << "]}";
|
||||
|
||||
ss << ", rtx: " << rtx.ToString();
|
||||
ss << ", c_name: " << c_name;
|
||||
@ -416,7 +416,7 @@ class VideoSendStreamImpl : public webrtc::BitrateAllocatorObserver,
|
||||
BitrateAllocator* const bitrate_allocator_;
|
||||
VieRemb* const remb_;
|
||||
|
||||
// TODO(brandtr): Consider moving this to a new FlexfecSendStream class.
|
||||
// TODO(brandtr): Move ownership to PayloadRouter.
|
||||
std::unique_ptr<FlexfecSender> flexfec_sender_;
|
||||
|
||||
rtc::CriticalSection ivf_writers_crit_;
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user