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

Unified Diff: webrtc/modules/audio_coding/neteq/tools/neteq_quality_test.h

Issue 1228843002: Update audio code to use size_t more correctly, (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Review comments Created 5 years, 4 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_coding/neteq/tools/neteq_quality_test.h
diff --git a/webrtc/modules/audio_coding/neteq/tools/neteq_quality_test.h b/webrtc/modules/audio_coding/neteq/tools/neteq_quality_test.h
index 4a0d808aeca8f2d802c8a48684aed351a85fbc6b..ba87dbf1f9d75602f09a05c2cf756bdd3bb6b506 100644
--- a/webrtc/modules/audio_coding/neteq/tools/neteq_quality_test.h
+++ b/webrtc/modules/audio_coding/neteq/tools/neteq_quality_test.h
@@ -76,8 +76,8 @@ class NetEqQualityTest : public ::testing::Test {
// |block_size_samples| (samples per channel),
// 2. save the bit stream to |payload| of |max_bytes| bytes in size,
// 3. returns the length of the payload (in bytes),
- virtual int EncodeBlock(int16_t* in_data, int block_size_samples,
- uint8_t* payload, int max_bytes) = 0;
+ virtual int EncodeBlock(int16_t* in_data, size_t block_size_samples,
+ uint8_t* payload, size_t max_bytes) = 0;
// PacketLost(...) determines weather a packet sent at an indicated time gets
// lost or not.
@@ -111,13 +111,13 @@ class NetEqQualityTest : public ::testing::Test {
const int out_sampling_khz_;
// Number of samples per channel in a frame.
- const int in_size_samples_;
+ const size_t in_size_samples_;
// Expected output number of samples per channel in a frame.
- const int out_size_samples_;
+ const size_t out_size_samples_;
size_t payload_size_bytes_;
- int max_payload_bytes_;
+ size_t max_payload_bytes_;
rtc::scoped_ptr<InputAudioFile> in_file_;
rtc::scoped_ptr<AudioSink> output_;

Powered by Google App Engine
This is Rietveld 408576698