commit 7bbd3b15fac5dad1ce32c4679d10b25e5738ce20
author Angie Chiang <angiebird@google.com> Wed Apr 26 11:06:09 2017 -0700
committer Angie Chiang <angiebird@google.com> Fri Apr 28 17:39:47 2017 +0000
tree 32cb2190e42d45ef55f3942403b5caf377bbfd20
parent df0e0d733f7602a47a9ba3aee6c1f772c790aee3

Add comment for RD_STATS

Change-Id: I30a6de3f7eb0c5ef3285ae62c39b3d14a94bd649

av1/common/blockd.h

diff --git a/av1/common/blockd.h b/av1/common/blockd.h
index 86a068e..8b80332 100644
--- a/av1/common/blockd.h
+++ b/av1/common/blockd.h
@@ -302,9 +302,14 @@
 typedef struct RD_STATS {
   int rate;
   int64_t dist;
+  // Please be careful of using rdcost, it's not guaranteed to be set all the
+  // time.
+  // TODO(angiebird): Create a set of functions to manipulate the RD_STATS. In
+  // these functions, make sure rdcost is always up-to-date according to
+  // rate/dist.
   int64_t rdcost;
   int64_t sse;
-  int skip;
+  int skip;  // sse should equal to dist when skip == 1
 #if CONFIG_RD_DEBUG
   int txb_coeff_cost[MAX_MB_PLANE];
 #if CONFIG_VAR_TX