| Index: webrtc/modules/audio_coding/neteq/neteq_impl_unittest.cc
|
| diff --git a/webrtc/modules/audio_coding/neteq/neteq_impl_unittest.cc b/webrtc/modules/audio_coding/neteq/neteq_impl_unittest.cc
|
| index 006a5ad542397cb839b2fe70bd5e59c093fbdbc8..f5d7814ad010c077fe58d74dfd7d527fe5f10e68 100644
|
| --- a/webrtc/modules/audio_coding/neteq/neteq_impl_unittest.cc
|
| +++ b/webrtc/modules/audio_coding/neteq/neteq_impl_unittest.cc
|
| @@ -470,14 +470,14 @@ TEST_F(NetEqImplTest, VerifyTimestampPropagation) {
|
| const size_t kMaxOutputSize = static_cast<size_t>(10 * kSampleRateHz / 1000);
|
| int16_t output[kMaxOutputSize];
|
| size_t samples_per_channel;
|
| - int num_channels;
|
| + size_t num_channels;
|
| NetEqOutputType type;
|
| EXPECT_EQ(
|
| NetEq::kOK,
|
| neteq_->GetAudio(
|
| kMaxOutputSize, output, &samples_per_channel, &num_channels, &type));
|
| ASSERT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(kOutputNormal, type);
|
|
|
| // Start with a simple check that the fake decoder is behaving as expected.
|
| @@ -550,14 +550,14 @@ TEST_F(NetEqImplTest, ReorderedPacket) {
|
| const size_t kMaxOutputSize = static_cast<size_t>(10 * kSampleRateHz / 1000);
|
| int16_t output[kMaxOutputSize];
|
| size_t samples_per_channel;
|
| - int num_channels;
|
| + size_t num_channels;
|
| NetEqOutputType type;
|
| EXPECT_EQ(
|
| NetEq::kOK,
|
| neteq_->GetAudio(
|
| kMaxOutputSize, output, &samples_per_channel, &num_channels, &type));
|
| ASSERT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(kOutputNormal, type);
|
|
|
| // Insert two more packets. The first one is out of order, and is already too
|
| @@ -590,7 +590,7 @@ TEST_F(NetEqImplTest, ReorderedPacket) {
|
| neteq_->GetAudio(
|
| kMaxOutputSize, output, &samples_per_channel, &num_channels, &type));
|
| ASSERT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(kOutputNormal, type);
|
|
|
| // Now check the packet buffer, and make sure it is empty, since the
|
| @@ -630,14 +630,14 @@ TEST_F(NetEqImplTest, FirstPacketUnknown) {
|
| const size_t kMaxOutputSize = static_cast<size_t>(10 * kSampleRateHz / 1000);
|
| int16_t output[kMaxOutputSize];
|
| size_t samples_per_channel;
|
| - int num_channels;
|
| + size_t num_channels;
|
| NetEqOutputType type;
|
| EXPECT_EQ(NetEq::kOK,
|
| neteq_->GetAudio(kMaxOutputSize, output, &samples_per_channel,
|
| &num_channels, &type));
|
| ASSERT_LE(samples_per_channel, kMaxOutputSize);
|
| EXPECT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(kOutputPLC, type);
|
|
|
| // Register the payload type.
|
| @@ -661,7 +661,7 @@ TEST_F(NetEqImplTest, FirstPacketUnknown) {
|
| &num_channels, &type));
|
| ASSERT_LE(samples_per_channel, kMaxOutputSize);
|
| EXPECT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(kOutputNormal, type)
|
| << "NetEq did not decode the packets as expected.";
|
| }
|
| @@ -744,7 +744,7 @@ TEST_F(NetEqImplTest, CodecInternalCng) {
|
| const size_t kMaxOutputSize = static_cast<size_t>(10 * kSampleRateKhz);
|
| int16_t output[kMaxOutputSize];
|
| size_t samples_per_channel;
|
| - int num_channels;
|
| + size_t num_channels;
|
| uint32_t timestamp;
|
| uint32_t last_timestamp;
|
| NetEqOutputType type;
|
| @@ -769,7 +769,7 @@ TEST_F(NetEqImplTest, CodecInternalCng) {
|
|
|
| for (size_t i = 1; i < 6; ++i) {
|
| ASSERT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(expected_type[i - 1], type);
|
| EXPECT_TRUE(neteq_->GetPlayoutTimestamp(×tamp));
|
| EXPECT_EQ(NetEq::kOK,
|
| @@ -790,7 +790,7 @@ TEST_F(NetEqImplTest, CodecInternalCng) {
|
|
|
| for (size_t i = 6; i < 8; ++i) {
|
| ASSERT_EQ(kMaxOutputSize, samples_per_channel);
|
| - EXPECT_EQ(1, num_channels);
|
| + EXPECT_EQ(1u, num_channels);
|
| EXPECT_EQ(expected_type[i - 1], type);
|
| EXPECT_EQ(NetEq::kOK,
|
| neteq_->GetAudio(kMaxOutputSize, output, &samples_per_channel,
|
| @@ -810,7 +810,7 @@ TEST_F(NetEqImplTest, UnsupportedDecoder) {
|
| UseNoMocks();
|
| CreateInstance();
|
| static const size_t kNetEqMaxFrameSize = 2880; // 60 ms @ 48 kHz.
|
| - static const int kChannels = 2;
|
| + static const size_t kChannels = 2;
|
|
|
| const uint8_t kPayloadType = 17; // Just an arbitrary number.
|
| const uint32_t kReceiveTime = 17; // Value doesn't matter for this test.
|
| @@ -886,11 +886,10 @@ TEST_F(NetEqImplTest, UnsupportedDecoder) {
|
| neteq_->InsertPacket(
|
| rtp_header, payload, kPayloadLengthBytes, kReceiveTime));
|
|
|
| - const size_t kMaxOutputSize =
|
| - static_cast<size_t>(10 * kSampleRateHz / 1000 * kChannels);
|
| + const size_t kMaxOutputSize = 10 * kSampleRateHz / 1000 * kChannels;
|
| int16_t output[kMaxOutputSize];
|
| size_t samples_per_channel;
|
| - int num_channels;
|
| + size_t num_channels;
|
| NetEqOutputType type;
|
|
|
| EXPECT_EQ(NetEq::kFail, neteq_->GetAudio(kMaxOutputSize, output,
|
|
|