Commit ef8eb291 authored by Dake He's avatar Dake He

[loopfilter] remove filter_length_internal

filter_length_internal seems redundant in the current implementation of
deblocking filter.

Change-Id: I40ada51857556c38cca56da33e17d0c2c82b8fc1
parent c048a2d9
...@@ -2032,8 +2032,6 @@ static TX_SIZE av1_get_transform_size( ...@@ -2032,8 +2032,6 @@ static TX_SIZE av1_get_transform_size(
typedef struct AV1_DEBLOCKING_PARAMETERS { typedef struct AV1_DEBLOCKING_PARAMETERS {
// length of the filter applied to the outer edge // length of the filter applied to the outer edge
uint32_t filter_length; uint32_t filter_length;
// length of the filter applied to the inner edge
uint32_t filter_length_internal;
// deblocking limits // deblocking limits
const uint8_t *lim; const uint8_t *lim;
const uint8_t *mblim; const uint8_t *mblim;
...@@ -2047,7 +2045,6 @@ static void set_lpf_parameters( ...@@ -2047,7 +2045,6 @@ static void set_lpf_parameters(
const struct macroblockd_plane *const plane_ptr) { const struct macroblockd_plane *const plane_ptr) {
// reset to initial values // reset to initial values
params->filter_length = 0; params->filter_length = 0;
params->filter_length_internal = 0;
// no deblocking is required // no deblocking is required
const uint32_t width = plane_ptr->dst.width; const uint32_t width = plane_ptr->dst.width;
...@@ -2171,7 +2168,7 @@ static void set_lpf_parameters( ...@@ -2171,7 +2168,7 @@ static void set_lpf_parameters(
} }
// prepare common parameters // 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; const loop_filter_thresh *const limits = cm->lf_info.lfthr + level;
params->lim = limits->lim; params->lim = limits->lim;
params->mblim = limits->mblim; params->mblim = limits->mblim;
...@@ -2261,16 +2258,6 @@ static void av1_filter_block_plane_vert( ...@@ -2261,16 +2258,6 @@ static void av1_filter_block_plane_vert(
// no filtering // no filtering
default: break; 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 // advance the destination pointer
p += MI_SIZE; p += MI_SIZE;
} }
...@@ -2363,16 +2350,6 @@ static void av1_filter_block_plane_horz( ...@@ -2363,16 +2350,6 @@ static void av1_filter_block_plane_horz(
// no filtering // no filtering
default: break; 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 // advance the destination pointer
p += MI_SIZE; p += MI_SIZE;
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment