Use size_t more consistently for packet/payload lengths.
See design doc at https://docs.google.com/a/chromium.org/document/d/1I6nmE9D_BmCY-IoV6MDPY2V6WYpEI-dg2apWXTfZyUI/edit?usp=sharing for more information.
This CL was reviewed and approved in pieces in the following CLs:
https://webrtc-codereview.appspot.com/24209004/
https://webrtc-codereview.appspot.com/24229004/
https://webrtc-codereview.appspot.com/24259004/
https://webrtc-codereview.appspot.com/25109004/
https://webrtc-codereview.appspot.com/26099004/
https://webrtc-codereview.appspot.com/27069004/
https://webrtc-codereview.appspot.com/27969004/
https://webrtc-codereview.appspot.com/27989004/
https://webrtc-codereview.appspot.com/29009004/
https://webrtc-codereview.appspot.com/30929004/
https://webrtc-codereview.appspot.com/30939004/
https://webrtc-codereview.appspot.com/31999004/
Committing as TBR to the original reviewers.
BUG=chromium:81439
TEST=none
TBR=pthatcher,henrik.lundin,tina.legrand,stefan,tkchin,glaznev,kjellander,perkj,mflodman,henrika,asapersson,niklas.enbom
Review URL: https://webrtc-codereview.appspot.com/23129004
git-svn-id: http://webrtc.googlecode.com/svn/trunk@7726 4adac7df-926f-26a2-2b94-8c16560cd09d
diff --git a/webrtc/modules/video_coding/codecs/test/stats.cc b/webrtc/modules/video_coding/codecs/test/stats.cc
index f6605f9..91a2f3c 100644
--- a/webrtc/modules/video_coding/codecs/test/stats.cc
+++ b/webrtc/modules/video_coding/codecs/test/stats.cc
@@ -15,6 +15,8 @@
#include <algorithm> // min_element, max_element
+#include "webrtc/base/format_macros.h"
+
namespace webrtc {
namespace test {
@@ -70,11 +72,11 @@
// Calculate min, max, average and total encoding time
int total_encoding_time_in_us = 0;
int total_decoding_time_in_us = 0;
- int total_encoded_frames_lengths = 0;
- int total_encoded_key_frames_lengths = 0;
- int total_encoded_nonkey_frames_lengths = 0;
- int nbr_keyframes = 0;
- int nbr_nonkeyframes = 0;
+ size_t total_encoded_frames_lengths = 0;
+ size_t total_encoded_key_frames_lengths = 0;
+ size_t total_encoded_nonkey_frames_lengths = 0;
+ size_t nbr_keyframes = 0;
+ size_t nbr_nonkeyframes = 0;
for (FrameStatisticsIterator it = stats_.begin();
it != stats_.end(); ++it) {
@@ -141,23 +143,24 @@
printf("Frame sizes:\n");
frame = std::min_element(stats_.begin(),
stats_.end(), LessForEncodedSize);
- printf(" Min : %7d bytes (frame %d)\n",
+ printf(" Min : %7" PRIuS " bytes (frame %d)\n",
frame->encoded_frame_length_in_bytes, frame->frame_number);
frame = std::max_element(stats_.begin(),
stats_.end(), LessForEncodedSize);
- printf(" Max : %7d bytes (frame %d)\n",
+ printf(" Max : %7" PRIuS " bytes (frame %d)\n",
frame->encoded_frame_length_in_bytes, frame->frame_number);
- printf(" Average : %7d bytes\n",
- static_cast<int>(total_encoded_frames_lengths / stats_.size()));
+ printf(" Average : %7" PRIuS " bytes\n",
+ total_encoded_frames_lengths / stats_.size());
if (nbr_keyframes > 0) {
- printf(" Average key frame size : %7d bytes (%d keyframes)\n",
- total_encoded_key_frames_lengths / nbr_keyframes,
- nbr_keyframes);
+ printf(" Average key frame size : %7" PRIuS " bytes (%" PRIuS
+ " keyframes)\n",
+ total_encoded_key_frames_lengths / nbr_keyframes, nbr_keyframes);
}
if (nbr_nonkeyframes > 0) {
- printf(" Average non-key frame size: %7d bytes (%d frames)\n",
+ printf(" Average non-key frame size: %7" PRIuS " bytes (%" PRIuS
+ " frames)\n",
total_encoded_nonkey_frames_lengths / nbr_nonkeyframes,
nbr_nonkeyframes);
}