ref: bf21ce63ee4dd1c539c3b437eb2bd04bf1c8767a
parent: c9f3f9ed70a448315f6d0300a9dd8c86f6ab7446
author: Jim Bankoski <jimbankoski@google.com>
date: Sat Oct 5 13:24:51 EDT 2013
encodemb cpplint issues revisited. Change-Id: Id5f25b74e2207bf44b6f6c8ffe548fa30fd78b4d
--- a/vp9/encoder/vp9_encodemb.c
+++ b/vp9/encoder/vp9_encodemb.c
@@ -80,8 +80,7 @@
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 @@
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 @@
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;
--
⑨