commit 70006e4681ea5a5eb46079a2b211377459466bf4
author Urvang Joshi <urvang@google.com> Wed Jun 14 16:08:55 2017 -0700
committer Urvang Joshi <urvang@google.com> Thu Jun 15 20:54:12 2017 +0000
tree e92ef2248951a2b6d33a4a00783ea0dc41f18d88
parent 0b99384644b6432902be2a0298ab3bbe60e6b00f

Remove 'rddiv' member from various structs.

This was initialized from a const and never modified. But was still
passed around and stored in multiple structs.

Removed these 'rddiv' member variables and now RDOPT() and RDOPT_DBL()
always use the const RDDIV_BITS directly.

Change-Id: I1a8dfd2c8fa857d466ad1207b4f0dd6ec07eafb8

av1/encoder/picklpf.c

diff --git a/av1/encoder/picklpf.c b/av1/encoder/picklpf.c
index fc0ea48..5d166d3 100644
--- a/av1/encoder/picklpf.c
+++ b/av1/encoder/picklpf.c
@@ -154,8 +154,7 @@
   // Update best error
   best_err = ss_err[filt_best];
 
-  if (best_cost_ret)
-    *best_cost_ret = RDCOST_DBL(x->rdmult, x->rddiv, 0, best_err);
+  if (best_cost_ret) *best_cost_ret = RDCOST_DBL(x->rdmult, 0, best_err);
   return filt_best;
 }