From: Yaowu Xu <yaowu@google.com>
Date: Fri, 16 May 2014 14:39:55 +0000 (-0700)
Subject: Add a TODO
X-Git-Tag: v1.4.0~1556^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c03ae7d99f5210e7fc970da2cd3906a1721f6c61;p=libvpx

Add a TODO

Change-Id: I16bf93d40e9b345705b49bf09dd4b6996b513a83
---

diff --git a/vp9/encoder/vp9_tokenize.c b/vp9/encoder/vp9_tokenize.c
index 17214c3ee..dcca92d21 100644
--- a/vp9/encoder/vp9_tokenize.c
+++ b/vp9/encoder/vp9_tokenize.c
@@ -288,7 +288,6 @@ struct is_skippable_args {
   MACROBLOCK *x;
   int *skippable;
 };
-
 static void is_skippable(int plane, int block,
                          BLOCK_SIZE plane_bsize, TX_SIZE tx_size,
                          void *argv) {
@@ -298,6 +297,8 @@ static void is_skippable(int plane, int block,
   args->skippable[0] &= (!args->x->plane[plane].eobs[block]);
 }
 
+// TODO(yaowu): rewrite and optimize this function to remove the usage of
+//              vp9_foreach_transform_block() and simplify is_skippable().
 int vp9_is_skippable_in_plane(MACROBLOCK *x, BLOCK_SIZE bsize, int plane) {
   int result = 1;
   struct is_skippable_args args = {x, &result};