commit 9814f15557ca0b658f19b6519722a82fb07bacec
author Frederic Barbier <frederic.barbier@allegrodvt.com> Wed Mar 13 11:47:17 2019 +0100
committer Yaowu Xu <yaowu@google.com> Fri Mar 15 21:39:27 2019 +0000
tree 2f85b77107ac090b07bed9aae409812c62b24739
parent f12c398b07792ca56fb76ce460fff026e4071d39

Cleanup dead code

Variable skip is never set to 1 and consequently is useless.

Change-Id: Ie52940aaf5c7f801ae6e45016c93c5b54e247343

av1/common/cdef.c

diff --git a/av1/common/cdef.c b/av1/common/cdef.c
index 556dede..63f9883 100644
--- a/av1/common/cdef.c
+++ b/av1/common/cdef.c
@@ -80,7 +80,6 @@
       if (!is_8x8_block_skip(grid, mi_row + r, mi_col + c, cm->mi_stride)) {
         dlist[count].by = r >> r_shift;
         dlist[count].bx = c >> c_shift;
-        dlist[count].skip = 0;
         count++;
       }
     }

av1/common/cdef_block.c

diff --git a/av1/common/cdef_block.c b/av1/common/cdef_block.c
index 845df37..dfd5882 100644
--- a/av1/common/cdef_block.c
+++ b/av1/common/cdef_block.c
@@ -232,8 +232,8 @@
   }
 
   for (bi = 0; bi < cdef_count; bi++) {
-    int t = dlist[bi].skip ? 0 : pri_strength;
-    int s = dlist[bi].skip ? 0 : sec_strength;
+    int t = pri_strength;
+    int s = sec_strength;
     by = dlist[bi].by;
     bx = dlist[bi].bx;
     if (dst8)

av1/common/cdef_block.h

diff --git a/av1/common/cdef_block.h b/av1/common/cdef_block.h
index 0e921e0..8321d48 100644
--- a/av1/common/cdef_block.h
+++ b/av1/common/cdef_block.h
@@ -38,7 +38,6 @@
 typedef struct {
   uint8_t by;
   uint8_t bx;
-  uint8_t skip;
 } cdef_list;
 
 typedef void (*cdef_filter_block_func)(uint8_t *dst8, uint16_t *dst16,