commit ef8eb291b828cfeb9980cf91e0984a773fcf8e5b
author Dake He <dkhe@google.com> Tue Jan 30 11:00:35 2018 -0800
committer Dake He <dkhe@google.com> Tue Jan 30 21:53:46 2018 +0000
tree 781c159e5610385ce67073e65ef60474754362e4
parent c048a2d97bb303b360ce904fc2fedc4a6d09d267

[loopfilter] remove filter_length_internal

filter_length_internal seems redundant in the current implementation of
deblocking filter.

Change-Id: I40ada51857556c38cca56da33e17d0c2c82b8fc1

av1/common/av1_loopfilter.c

diff --git a/av1/common/av1_loopfilter.c b/av1/common/av1_loopfilter.c
index 1732cb6..a6a8ef4 100644
--- a/av1/common/av1_loopfilter.c
+++ b/av1/common/av1_loopfilter.c
@@ -2032,8 +2032,6 @@
 typedef struct AV1_DEBLOCKING_PARAMETERS {
   // length of the filter applied to the outer edge
   uint32_t filter_length;
-  // length of the filter applied to the inner edge
-  uint32_t filter_length_internal;
   // deblocking limits
   const uint8_t *lim;
   const uint8_t *mblim;
@@ -2047,7 +2045,6 @@
     const struct macroblockd_plane *const plane_ptr) {
   // reset to initial values
   params->filter_length = 0;
-  params->filter_length_internal = 0;
 
   // no deblocking is required
   const uint32_t width = plane_ptr->dst.width;
@@ -2171,7 +2168,7 @@
       }
 
       // prepare common parameters
-      if (params->filter_length || params->filter_length_internal) {
+      if (params->filter_length) {
         const loop_filter_thresh *const limits = cm->lf_info.lfthr + level;
         params->lim = limits->lim;
         params->mblim = limits->mblim;
@@ -2261,16 +2258,6 @@
         // no filtering
         default: break;
       }
-      // process the internal edge
-      if (params.filter_length_internal) {
-        if (cm->use_highbitdepth)
-          aom_highbd_lpf_vertical_4(CONVERT_TO_SHORTPTR(p + 4), dst_stride,
-                                    params.mblim, params.lim, params.hev_thr,
-                                    cm->bit_depth);
-        else
-          aom_lpf_vertical_4(p + 4, dst_stride, params.mblim, params.lim,
-                             params.hev_thr);
-      }
       // advance the destination pointer
       p += MI_SIZE;
     }
@@ -2363,16 +2350,6 @@
         // no filtering
         default: break;
       }
-      // process the internal edge
-      if (params.filter_length_internal) {
-        if (cm->use_highbitdepth)
-          aom_highbd_lpf_horizontal_4(CONVERT_TO_SHORTPTR(p + 4 * dst_stride),
-                                      dst_stride, params.mblim, params.lim,
-                                      params.hev_thr, cm->bit_depth);
-        else
-          aom_lpf_horizontal_4(p + 4 * dst_stride, dst_stride, params.mblim,
-                               params.lim, params.hev_thr);
-      }
       // advance the destination pointer
       p += MI_SIZE;
     }