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

Side by Side Diff: webrtc/modules/audio_coding/include/audio_coding_module.h

Issue 2286063005: Fix Chromium clang plugin warnings (Closed)
Patch Set: Created 4 years, 3 months 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 unified diff | Download patch
« no previous file with comments | « webrtc/modules/audio_coding/acm2/audio_coding_module.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 57
58 virtual int32_t InFrameType(FrameType frame_type) = 0; 58 virtual int32_t InFrameType(FrameType frame_type) = 0;
59 }; 59 };
60 60
61 class AudioCodingModule { 61 class AudioCodingModule {
62 protected: 62 protected:
63 AudioCodingModule() {} 63 AudioCodingModule() {}
64 64
65 public: 65 public:
66 struct Config { 66 struct Config {
67 Config() : id(0), neteq_config(), clock(Clock::GetRealTimeClock()) { 67 Config();
68 // Post-decode VAD is disabled by default in NetEq, however, Audio 68 Config(const Config&);
69 // Conference Mixer relies on VAD decisions and fails without them. 69 ~Config();
70 neteq_config.enable_post_decode_vad = true;
71 }
72 70
73 int id; 71 int id;
74 NetEq::Config neteq_config; 72 NetEq::Config neteq_config;
75 Clock* clock; 73 Clock* clock;
76 rtc::scoped_refptr<AudioDecoderFactory> decoder_factory; 74 rtc::scoped_refptr<AudioDecoderFactory> decoder_factory;
77 }; 75 };
78 76
79 /////////////////////////////////////////////////////////////////////////// 77 ///////////////////////////////////////////////////////////////////////////
80 // Creation and destruction of a ACM. 78 // Creation and destruction of a ACM.
81 // 79 //
(...skipping 743 matching lines...) Expand 10 before | Expand all | Expand 10 after
825 virtual std::vector<uint16_t> GetNackList( 823 virtual std::vector<uint16_t> GetNackList(
826 int64_t round_trip_time_ms) const = 0; 824 int64_t round_trip_time_ms) const = 0;
827 825
828 virtual void GetDecodingCallStatistics( 826 virtual void GetDecodingCallStatistics(
829 AudioDecodingCallStats* call_stats) const = 0; 827 AudioDecodingCallStats* call_stats) const = 0;
830 }; 828 };
831 829
832 } // namespace webrtc 830 } // namespace webrtc
833 831
834 #endif // WEBRTC_MODULES_AUDIO_CODING_INCLUDE_AUDIO_CODING_MODULE_H_ 832 #endif // WEBRTC_MODULES_AUDIO_CODING_INCLUDE_AUDIO_CODING_MODULE_H_
OLDNEW
« no previous file with comments | « webrtc/modules/audio_coding/acm2/audio_coding_module.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698