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

Unified Diff: webrtc/modules/audio_processing/vad/vad_audio_proc.h

Issue 2320053003: webrtc/modules/audio_processing: Use RTC_DCHECK() instead of assert() (Closed)
Patch Set: rebase 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_processing/vad/vad_audio_proc.h
diff --git a/webrtc/modules/audio_processing/vad/vad_audio_proc.h b/webrtc/modules/audio_processing/vad/vad_audio_proc.h
index a8ecc7ba329e046a10806de0fa3e807e4068917e..1f27b294e552398133589012a7f98cd24b06d388 100644
--- a/webrtc/modules/audio_processing/vad/vad_audio_proc.h
+++ b/webrtc/modules/audio_processing/vad/vad_audio_proc.h
@@ -50,25 +50,28 @@ class VadAudioProc {
// For every 30 ms we compute 3 spectral peak there for 3 LPC analysis.
// LPC is computed over 15 ms of windowed audio. For every 10 ms sub-frame
// we need 5 ms of past signal to create the input of LPC analysis.
- static const size_t kNumPastSignalSamples =
- static_cast<size_t>(kSampleRateHz / 200);
+ enum : size_t {
+ kNumPastSignalSamples = static_cast<size_t>(kSampleRateHz / 200)
+ };
// TODO(turajs): maybe defining this at a higher level (maybe enum) so that
// all the code recognize it as "no-error."
- static const int kNoError = 0;
-
- static const size_t kNum10msSubframes = 3;
- static const size_t kNumSubframeSamples =
- static_cast<size_t>(kSampleRateHz / 100);
- static const size_t kNumSamplesToProcess =
- kNum10msSubframes *
- kNumSubframeSamples; // Samples in 30 ms @ given sampling rate.
- static const size_t kBufferLength =
- kNumPastSignalSamples + kNumSamplesToProcess;
- static const size_t kIpLength = kDftSize >> 1;
- static const size_t kWLength = kDftSize >> 1;
-
- static const size_t kLpcOrder = 16;
+ enum : int { kNoError = 0 };
+
+ enum : size_t { kNum10msSubframes = 3 };
+ enum : size_t {
+ kNumSubframeSamples = static_cast<size_t>(kSampleRateHz / 100)
+ };
+ enum : size_t {
+ // Samples in 30 ms @ given sampling rate.
+ kNumSamplesToProcess = kNum10msSubframes * kNumSubframeSamples
+ };
+ enum : size_t {
+ kBufferLength = kNumPastSignalSamples + kNumSamplesToProcess
+ };
+ enum : size_t { kIpLength = kDftSize >> 1 };
+ enum : size_t { kWLength = kDftSize >> 1 };
+ enum : size_t { kLpcOrder = 16 };
size_t ip_[kIpLength];
float w_fft_[kWLength];
« no previous file with comments | « webrtc/modules/audio_processing/vad/standalone_vad.cc ('k') | webrtc/modules/audio_processing/vad/vad_audio_proc.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698