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

Side by Side Diff: webrtc/common_audio/vad/vad.cc

Issue 2623313004: Replace RTC_DCHECK(false) with RTC_NOTREACHED(). (Closed)
Patch Set: Created 3 years, 11 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/base/virtualsocketserver.cc ('k') | webrtc/media/engine/webrtcvideoengine2.cc » ('j') | 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) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 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 19 matching lines...) Expand all
30 Activity VoiceActivity(const int16_t* audio, 30 Activity VoiceActivity(const int16_t* audio,
31 size_t num_samples, 31 size_t num_samples,
32 int sample_rate_hz) override { 32 int sample_rate_hz) override {
33 int ret = WebRtcVad_Process(handle_, sample_rate_hz, audio, num_samples); 33 int ret = WebRtcVad_Process(handle_, sample_rate_hz, audio, num_samples);
34 switch (ret) { 34 switch (ret) {
35 case 0: 35 case 0:
36 return kPassive; 36 return kPassive;
37 case 1: 37 case 1:
38 return kActive; 38 return kActive;
39 default: 39 default:
40 RTC_DCHECK(false) << "WebRtcVad_Process returned an error."; 40 RTC_NOTREACHED() << "WebRtcVad_Process returned an error.";
41 return kError; 41 return kError;
42 } 42 }
43 } 43 }
44 44
45 void Reset() override { 45 void Reset() override {
46 if (handle_) 46 if (handle_)
47 WebRtcVad_Free(handle_); 47 WebRtcVad_Free(handle_);
48 handle_ = WebRtcVad_Create(); 48 handle_ = WebRtcVad_Create();
49 RTC_CHECK(handle_); 49 RTC_CHECK(handle_);
50 RTC_CHECK_EQ(WebRtcVad_Init(handle_), 0); 50 RTC_CHECK_EQ(WebRtcVad_Init(handle_), 0);
51 RTC_CHECK_EQ(WebRtcVad_set_mode(handle_, aggressiveness_), 0); 51 RTC_CHECK_EQ(WebRtcVad_set_mode(handle_, aggressiveness_), 0);
52 } 52 }
53 53
54 private: 54 private:
55 VadInst* handle_; 55 VadInst* handle_;
56 Aggressiveness aggressiveness_; 56 Aggressiveness aggressiveness_;
57 }; 57 };
58 58
59 } // namespace 59 } // namespace
60 60
61 std::unique_ptr<Vad> CreateVad(Vad::Aggressiveness aggressiveness) { 61 std::unique_ptr<Vad> CreateVad(Vad::Aggressiveness aggressiveness) {
62 return std::unique_ptr<Vad>(new VadImpl(aggressiveness)); 62 return std::unique_ptr<Vad>(new VadImpl(aggressiveness));
63 } 63 }
64 64
65 } // namespace webrtc 65 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/base/virtualsocketserver.cc ('k') | webrtc/media/engine/webrtcvideoengine2.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698