[NOT FOR REVIEW] Various changes requested by Andrew

webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c

Index: webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c
diff --git a/webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c b/webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c
index 66dfafbe7a436b2e4283f9a7874ff37449a772ff..ad095da0b492f946ea5b7e8efa1ab51d0d14ca13 100644
--- a/webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c
+++ b/webrtc/modules/audio_coding/codecs/ilbc/get_sync_seq.c
@@ -27,52 +27,52 @@
 
 void WebRtcIlbcfix_GetSyncSeq(
     int16_t *idata,   /* (i) original data */
-    int16_t idatal,   /* (i) dimension of data */
-    int16_t centerStartPos, /* (i) where current block starts */
-    int16_t *period,   /* (i) rough-pitch-period array       (Q-2) */
-    int16_t *plocs,   /* (i) where periods of period array are taken (Q-2) */
+    size_t idatal,   /* (i) dimension of data */
+    size_t centerStartPos, /* (i) where current block starts */
+    size_t *period,   /* (i) rough-pitch-period array       (Q-2) */
+    const size_t *plocs, /* (i) where periods of period array are taken (Q-2) */
     size_t periodl,   /* (i) dimension period array */
-    int16_t hl,    /* (i) 2*hl+1 is the number of sequences */
+    size_t hl,    /* (i) 2*hl+1 is the number of sequences */
     int16_t *surround  /* (i/o) The contribution from this sequence
                                 summed with earlier contributions */
                               ){
-  size_t i;
-  int16_t centerEndPos,q;
+  size_t i, centerEndPos, q;
   /* Stack based */
-  int16_t lagBlock[2*ENH_HL+1];
-  int16_t blockStartPos[2*ENH_HL+1]; /* Defines the position to search around (Q2) */
-  int16_t plocs2[ENH_PLOCSL];
+  size_t lagBlock[2 * ENH_HL + 1];
+  size_t blockStartPos[2 * ENH_HL + 1]; /* The position to search around (Q2) */
+  size_t plocs2[ENH_PLOCSL];
 
-  centerEndPos=centerStartPos+ENH_BLOCKL-1;
+  centerEndPos = centerStartPos + ENH_BLOCKL - 1;
 
   /* present (find predicted lag from this position) */
 
   WebRtcIlbcfix_NearestNeighbor(lagBlock + hl,
                                 plocs,
-                                (int16_t)(2 * (centerStartPos + centerEndPos)),
+                                2 * (centerStartPos + centerEndPos),
                                 periodl);
 
-  blockStartPos[hl] = (int16_t)(4 * centerStartPos);
+  blockStartPos[hl] = 4 * centerStartPos;
 
   /* past (find predicted position and perform a refined
      search to find the best sequence) */
 
-  for(q=hl-1;q>=0;q--) {
-    blockStartPos[q]=blockStartPos[q+1]-period[lagBlock[q+1]];
+  for (q = hl; q > 0; q--) {
+    size_t qq = q - 1;
+    blockStartPos[qq] = blockStartPos[q] - period[lagBlock[q]];
 
     WebRtcIlbcfix_NearestNeighbor(
-        lagBlock + q,
+        lagBlock + qq,
         plocs,
-        (int16_t)(blockStartPos[q] + 4 * ENH_BLOCKL_HALF -
-            period[lagBlock[q + 1]]),
+        blockStartPos[qq] + 4 * ENH_BLOCKL_HALF - period[lagBlock[q]],
         periodl);
 
-    if (blockStartPos[q] - 4 * ENH_OVERHANG >= 0) {
+    if (blockStartPos[qq] >= (4 * ENH_OVERHANG)) {
 
       /* Find the best possible sequence in the 4 times upsampled
          domain around blockStartPos+q */
-      WebRtcIlbcfix_Refiner(blockStartPos+q,idata,idatal,
-                            centerStartPos,blockStartPos[q],surround,WebRtcIlbcfix_kEnhWt[q]);
+      WebRtcIlbcfix_Refiner(blockStartPos + qq, idata, idatal, centerStartPos,
+                            blockStartPos[qq], surround,
+                            WebRtcIlbcfix_kEnhWt[qq]);
 
     } else {
       /* Don't add anything since this sequence would
@@ -83,29 +83,29 @@ void WebRtcIlbcfix_GetSyncSeq(
   /* future (find predicted position and perform a refined
      search to find the best sequence) */
 
-  for(i=0;i<periodl;i++) {
-    plocs2[i]=(plocs[i]-period[i]);
+  for (i = 0; i < periodl; i++) {
+    plocs2[i] = plocs[i] - period[i];
   }
 
-  for (q = hl + 1; q <= (int16_t)(2 * hl); q++) {
+  for (q = hl + 1; q <= (2 * hl); q++) {
 
     WebRtcIlbcfix_NearestNeighbor(
         lagBlock + q,
         plocs2,
-        (int16_t)(blockStartPos[q - 1] + 4 * ENH_BLOCKL_HALF),
+        blockStartPos[q - 1] + 4 * ENH_BLOCKL_HALF,
         periodl);
 
     blockStartPos[q]=blockStartPos[q-1]+period[lagBlock[q]];
 
-    if (blockStartPos[q] + 4 * (ENH_BLOCKL + ENH_OVERHANG) < 4 * idatal) {
+    if (blockStartPos[q] < (4 * (idatal + ENH_BLOCKL + ENH_OVERHANG))) {
 
       /* Find the best possible sequence in the 4 times upsampled
          domain around blockStartPos+q */
-      WebRtcIlbcfix_Refiner(blockStartPos+q, idata, idatal,
-                            centerStartPos,blockStartPos[q],surround,WebRtcIlbcfix_kEnhWt[2*hl-q]);
+      WebRtcIlbcfix_Refiner(blockStartPos + q, idata, idatal, centerStartPos,
+                            blockStartPos[q], surround,
+                            WebRtcIlbcfix_kEnhWt[2 * hl - q]);
 
-    }
-    else {
+    } else {
       /* Don't add anything since this sequence would
          be outside the buffer */
     }