Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h

Issue 2392883002: Multi frequency DTMF support - sender side (Closed)
Patch Set: rebase Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h b/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h
index aaee73956ca59639fa855ec359fb209edbf63e05..ee4831c6348c5277efa7632cee974fb5a376f65a 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h
@@ -57,7 +57,6 @@ class RTPSenderAudio {
protected:
bool SendTelephoneEventPacket(
bool ended,
- int8_t dtmf_payload_type,
uint32_t dtmf_timestamp,
uint16_t duration,
bool marker_bit); // set on first packet in talk burst
@@ -65,36 +64,36 @@ class RTPSenderAudio {
bool MarkerBit(FrameType frame_type, int8_t payload_type);
private:
- Clock* const clock_;
- RTPSender* const rtp_sender_;
+ Clock* const clock_ = nullptr;
+ RTPSender* const rtp_sender_ = nullptr;
rtc::CriticalSection send_audio_critsect_;
- uint16_t packet_size_samples_ GUARDED_BY(send_audio_critsect_);
+ uint16_t packet_size_samples_ GUARDED_BY(send_audio_critsect_) = 160;
// DTMF.
- bool dtmf_event_is_on_;
- bool dtmf_event_first_packet_sent_;
- int8_t dtmf_payload_type_ GUARDED_BY(send_audio_critsect_);
- uint32_t dtmf_timestamp_;
- uint8_t dtmf_key_;
- uint32_t dtmf_length_samples_;
- uint8_t dtmf_level_;
- int64_t dtmf_time_last_sent_;
- uint32_t dtmf_timestamp_last_sent_;
- DTMFqueue dtmf_queue_;
+ bool dtmf_event_is_on_ = false;
+ bool dtmf_event_first_packet_sent_ = false;
+ int8_t dtmf_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
+ uint32_t dtmf_payload_freq_ GUARDED_BY(send_audio_critsect_) = 8000;
+ uint32_t dtmf_timestamp_ = 0;
+ uint32_t dtmf_length_samples_ = 0;
+ int64_t dtmf_time_last_sent_ = 0;
+ uint32_t dtmf_timestamp_last_sent_ = 0;
+ DtmfQueue::Event dtmf_current_event_;
+ DtmfQueue dtmf_queue_;
// VAD detection, used for marker bit.
- bool inband_vad_active_ GUARDED_BY(send_audio_critsect_);
- int8_t cngnb_payload_type_ GUARDED_BY(send_audio_critsect_);
- int8_t cngwb_payload_type_ GUARDED_BY(send_audio_critsect_);
- int8_t cngswb_payload_type_ GUARDED_BY(send_audio_critsect_);
- int8_t cngfb_payload_type_ GUARDED_BY(send_audio_critsect_);
- int8_t last_payload_type_ GUARDED_BY(send_audio_critsect_);
+ bool inband_vad_active_ GUARDED_BY(send_audio_critsect_) = false;
+ int8_t cngnb_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
+ int8_t cngwb_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
+ int8_t cngswb_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
+ int8_t cngfb_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
+ int8_t last_payload_type_ GUARDED_BY(send_audio_critsect_) = -1;
// Audio level indication.
// (https://datatracker.ietf.org/doc/draft-lennox-avt-rtp-audio-level-exthdr/)
- uint8_t audio_level_dbov_ GUARDED_BY(send_audio_critsect_);
+ uint8_t audio_level_dbov_ GUARDED_BY(send_audio_critsect_) = 0;
OneTimeEvent first_packet_sent_;
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(RTPSenderAudio);
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_rtcp_config.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698