Reformat existing code. There should be no functional effects.

webrtc/modules/audio_coding/codecs/g711/test/testG711.cc

Index: webrtc/modules/audio_coding/codecs/g711/test/testG711.cc
diff --git a/webrtc/modules/audio_coding/codecs/g711/test/testG711.cc b/webrtc/modules/audio_coding/codecs/g711/test/testG711.cc
index e891810ca957d3ea82b641cec74d926bf7a546e3..a246ae8badc487a23b50588bd3e82f4160fc58ee 100644
--- a/webrtc/modules/audio_coding/codecs/g711/test/testG711.cc
+++ b/webrtc/modules/audio_coding/codecs/g711/test/testG711.cc
@@ -24,18 +24,12 @@
 #define CLOCKS_PER_SEC_G711 1000
 
 /* function for reading audio data from PCM file */
-int readframe(int16_t* data, FILE* inp, int length) {
-
-  short k, rlen, status = 0;
-
-  rlen = (short) fread(data, sizeof(int16_t), length, inp);
-  if (rlen < length) {
-    for (k = rlen; k < length; k++)
-      data[k] = 0;
-    status = 1;
-  }
-
-  return status;
+bool readframe(int16_t* data, FILE* inp, int length) {
+  short rlen = (short) fread(data, sizeof(int16_t), length, inp);
+  if (rlen >= length)
+    return false;
+  memset(data + rlen, 0, (length - rlen) * sizeof(int16_t));
+  return true;

[kwiberg-webrtc, 2015/06/10 11:58:32]

Umm... this return true if it failed and false if it succeeded? Please add that
to the comment...

Oh, and put it in an anonymous namespace while you're at it.

 }
 
 int main(int argc, char* argv[]) {
@@ -43,7 +37,8 @@ int main(int argc, char* argv[]) {
   FILE* inp;
   FILE* outp;
   FILE* bitp = NULL;
-  int framecnt, endfile;
+  int framecnt;
+  bool endfile;

[kwiberg-webrtc, 2015/06/10 11:58:32]

Consider moving declarations to first use while you're at it.

 
   int16_t framelength = 80;
 
@@ -118,8 +113,8 @@ int main(int argc, char* argv[]) {
 
   /* Initialize encoder and decoder */
   framecnt = 0;
-  endfile = 0;
-  while (endfile == 0) {
+  endfile = false;
+  while (!endfile) {
     framecnt++;
     /* Read speech block */
     endfile = readframe(shortdata, inp, framelength);