]> granicus.if.org Git - libvpx/commitdiff
encodemb cpplint issues revisited.
authorJim Bankoski <jimbankoski@google.com>
Sun, 6 Oct 2013 00:24:51 +0000 (17:24 -0700)
committerJim Bankoski <jimbankoski@google.com>
Sun, 6 Oct 2013 00:24:51 +0000 (17:24 -0700)
Change-Id: Id5f25b74e2207bf44b6f6c8ffe548fa30fd78b4d

vp9/encoder/vp9_encodemb.c

index a610d634083704003f3cf549a8252017cc4e1141..1f36f161865f4ffbe2594eb476ca39e85a6b81b8 100644 (file)
@@ -80,8 +80,7 @@ void vp9_subtract_sb(MACROBLOCK *x, BLOCK_SIZE bsize) {
   vp9_subtract_sbuv(x, bsize);
 }
 
-
-#define RDTRUNC(RM,DM,R,D) ( (128+(R)*(RM)) & 0xFF )
+#define RDTRUNC(RM, DM, R, D) ((128 + (R) * (RM)) & 0xFF)
 typedef struct vp9_token_state vp9_token_state;
 
 struct vp9_token_state {
@@ -92,7 +91,7 @@ struct vp9_token_state {
   short         qc;
 };
 
-// TODO: experiments to find optimal multiple numbers
+// TODO(jimbankoski): experiment to find optimal RD numbers.
 #define Y1_RD_MULT 4
 #define UV_RD_MULT 2
 
@@ -272,11 +271,10 @@ static void optimize_b(MACROBLOCK *mb,
       best_index[i][1] = best;
       /* Finally, make this the new head of the trellis. */
       next = i;
-    }
-    /* There's no choice to make for a zero coefficient, so we don't
-     *  add a new trellis node, but we do need to update the costs.
-     */
-    else {
+    } else {
+      /* There's no choice to make for a zero coefficient, so we don't
+       *  add a new trellis node, but we do need to update the costs.
+       */
       band = get_coef_band(band_translate, i + 1);
       t0 = tokens[next][0].token;
       t1 = tokens[next][1].token;