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

Side by Side Diff: webrtc/modules/audio_processing/include/audio_processing.h

Issue 1338833002: Fix the maximum native sample rate in AudioProcessing (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 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 | « no previous file | 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 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 // AudioProcessing::SetExtraOptions(). 121 // AudioProcessing::SetExtraOptions().
122 // 122 //
123 // Note: If enabled and the reverse stream has more than one output channel, 123 // Note: If enabled and the reverse stream has more than one output channel,
124 // the reverse stream will become an upmixed mono signal. 124 // the reverse stream will become an upmixed mono signal.
125 struct Intelligibility { 125 struct Intelligibility {
126 Intelligibility() : enabled(false) {} 126 Intelligibility() : enabled(false) {}
127 explicit Intelligibility(bool enabled) : enabled(enabled) {} 127 explicit Intelligibility(bool enabled) : enabled(enabled) {}
128 bool enabled; 128 bool enabled;
129 }; 129 };
130 130
131 static const int kAudioProcMaxNativeSampleRateHz = 32000; 131 static const int kAudioProcMaxNativeSampleRateHz = 48000;
Andrew MacDonald 2015/09/11 18:06:23 Looks like this should only affect: https://code.g
aluebs-webrtc 2015/09/15 22:56:09 You were right, it was not working for 44100, than
132 132
133 // The Audio Processing Module (APM) provides a collection of voice processing 133 // The Audio Processing Module (APM) provides a collection of voice processing
134 // components designed for real-time communications software. 134 // components designed for real-time communications software.
135 // 135 //
136 // APM operates on two audio streams on a frame-by-frame basis. Frames of the 136 // APM operates on two audio streams on a frame-by-frame basis. Frames of the
137 // primary stream, on which all processing is applied, are passed to 137 // primary stream, on which all processing is applied, are passed to
138 // |ProcessStream()|. Frames of the reverse direction stream, which are used for 138 // |ProcessStream()|. Frames of the reverse direction stream, which are used for
139 // analysis by some components, are passed to |AnalyzeReverseStream()|. On the 139 // analysis by some components, are passed to |AnalyzeReverseStream()|. On the
140 // client-side, this will typically be the near-end (capture) and far-end 140 // client-side, this will typically be the near-end (capture) and far-end
141 // (render) streams, respectively. APM should be placed in the signal chain as 141 // (render) streams, respectively. APM should be placed in the signal chain as
(...skipping 794 matching lines...) Expand 10 before | Expand all | Expand 10 after
936 // This does not impact the size of frames passed to |ProcessStream()|. 936 // This does not impact the size of frames passed to |ProcessStream()|.
937 virtual int set_frame_size_ms(int size) = 0; 937 virtual int set_frame_size_ms(int size) = 0;
938 virtual int frame_size_ms() const = 0; 938 virtual int frame_size_ms() const = 0;
939 939
940 protected: 940 protected:
941 virtual ~VoiceDetection() {} 941 virtual ~VoiceDetection() {}
942 }; 942 };
943 } // namespace webrtc 943 } // namespace webrtc
944 944
945 #endif // WEBRTC_MODULES_AUDIO_PROCESSING_INCLUDE_AUDIO_PROCESSING_H_ 945 #endif // WEBRTC_MODULES_AUDIO_PROCESSING_INCLUDE_AUDIO_PROCESSING_H_
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698