Update audio code to use size_t more correctly, webrtc/common_audio/ portion.

webrtc/common_audio/vad/vad_unittest.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #include "webrtc/common_audio/vad/vad_unittest.h"
 
 #include <stdlib.h>
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include "webrtc/base/arraysize.h"
 #include "webrtc/base/checks.h"
 #include "webrtc/common_audio/signal_processing/include/signal_processing_library.h"
 #include "webrtc/common_audio/vad/include/webrtc_vad.h"
 #include "webrtc/typedefs.h"
 
 VadTest::VadTest() {}
 
 void VadTest::SetUp() {}
 
 void VadTest::TearDown() {}
 
 // Returns true if the rate and frame length combination is valid.
-bool VadTest::ValidRatesAndFrameLengths(int rate, int frame_length) {
+bool VadTest::ValidRatesAndFrameLengths(int rate, size_t frame_length) {
   if (rate == 8000) {
     if (frame_length == 80 || frame_length == 160 || frame_length == 240) {
       return true;
     }
     return false;
   } else if (rate == 16000) {
     if (frame_length == 160 || frame_length == 320 || frame_length == 480) {
       return true;
     }
     return false;
@@ skipping 17 matching lines @@
 TEST_F(VadTest, ApiTest) {
   // This API test runs through the APIs for all possible valid and invalid
   // combinations.
 
   VadInst* handle = WebRtcVad_Create();
   int16_t zeros[kMaxFrameLength] = { 0 };
 
   // Construct a speech signal that will trigger the VAD in all modes. It is
   // known that (i * i) will wrap around, but that doesn't matter in this case.
   int16_t speech[kMaxFrameLength];
-  for (int16_t i = 0; i < kMaxFrameLength; i++) {
+  for (size_t i = 0; i < kMaxFrameLength; i++) {
     speech[i] = static_cast<int16_t>(i * i);
   }
 
   // nullptr instance tests
   EXPECT_EQ(-1, WebRtcVad_Init(nullptr));
   EXPECT_EQ(-1, WebRtcVad_set_mode(nullptr, kModes[0]));
   EXPECT_EQ(-1,
             WebRtcVad_Process(nullptr, kRates[0], speech, kFrameLengths[0]));
 
   // WebRtcVad_Create()
@@ skipping 48 matching lines @@
 }
 
 TEST_F(VadTest, ValidRatesFrameLengths) {
   // This test verifies valid and invalid rate/frame_length combinations. We
   // loop through some sampling rates and frame lengths from negative values to
   // values larger than possible.
   const int kRates[] = {
     -8000, -4000, 0, 4000, 8000, 8001, 15999, 16000, 32000, 48000, 48001, 96000
   };
 
-  const int kFrameLengths[] = {
-    -10, 0, 80, 81, 159, 160, 240, 320, 480, 640, 960, 1440, 2000
+  const size_t kFrameLengths[] = {
+    0, 80, 81, 159, 160, 240, 320, 480, 640, 960, 1440, 2000
   };
 
   for (size_t i = 0; i < arraysize(kRates); i++) {
     for (size_t j = 0; j < arraysize(kFrameLengths); j++) {
       if (ValidRatesAndFrameLengths(kRates[i], kFrameLengths[j])) {
         EXPECT_EQ(0, WebRtcVad_ValidRateAndFrameLength(kRates[i],
                                                        kFrameLengths[j]));
       } else {
         EXPECT_EQ(-1, WebRtcVad_ValidRateAndFrameLength(kRates[i],
                                                         kFrameLengths[j]));
       }
     }
   }
 }
 
 // TODO(bjornv): Add a process test, run on file.
 
 }  // namespace